Skip to content

ATL-1063: Integrated search in workspace into IDE #202

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 15, 2021

Conversation

kittaakos
Copy link
Contributor

Signed-off-by: Akos Kitta kittaakos@typefox.io

What it does:

  • This PR integrates Theia's default Search view into the IDE2.
  • From now on, the Search view is part of the default layout, so when the user opens a new sketch, the Search view (with the magnifier icon. the monitor widget has a similar icon 😕) should be visible on the left sidebar.
  • This PR changes the order of the widgets on the left: Board Manager, Lib Manager, Debug, Explorer (this is hidden), Search, and Outline is the order. Then the rest of the views, if any.
  • This PR changes the default width of the sidebar views. When you open the view, it should require 200px width. It was 100px before. I thought increasing the width is a good thing, whenever I open a view, I have to change the horizontal dimension every time manually. What do you think?

Please review. Thank you!

@ubidefeo
Copy link

hi Akos

This PR integrates Theia's default Search view into the IDE2.
From now on, the Search view is part of the default layout, so when the user opens a new sketch, the Search view (with the magnifier icon. the monitor widget has a similar icon 😕) should be visible on the left sidebar.

This is a welcome addition to the IDE. Don't worry about the icon, we'll have to go through a final polishing of the theme at some point

This PR changes the order of the widgets on the left: Board Manager, Lib Manager, Debug, Explorer (this is hidden), Search, and Outline is the order. Then the rest of the views, if any.

I toggled the Explorer and it comes before the Debugger
Screenshot 2021-03-11 at 22 08 38

This PR changes the default width of the sidebar views. When you open the view, it should require 200px width. It was 100px before. I thought increasing the width is a good thing, whenever I open a view, I have to change the horizontal dimension every time manually. What do you think?

200px is a better option. I ran into the same issue you did and had to manually resize.
It was gonna turn into a task at some point :D

I'm fine with everything except the position of the Explorer if toggled. I'd place this at the very top, VS Code style ;)

@kittaakos
Copy link
Contributor Author

Explorer if toggled. I'd place this at the very top, VS Code style ;)

👍 Even better. Thank you for the review.

Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, here I am again being a pain with my dirty sketches.

  1. File > Preferences
  2. Uncheck the box next to "Auto save".
  3. Click the "OK" button.
  4. Add the following text to a sketch: foobar
  5. Click the search icon in the sidebar.
  6. In the "Search" field", type bar
  7. Click the downward pointing triangle button at the bottom left corner of "Search" field to reveal the "Replace" field.
  8. In the "Replace" field", type baz - in the results section, the expected "foobarbaz" is shown.
  9. Click "foobarbaz" to open the "Replace Preview" view in the editor. - the right side editor pane only shows "baz", rather than the expected "foobaz"

Clipboard01

@kittaakos kittaakos force-pushed the atl-1063--multi-tab-search branch from 84459f9 to 58d0a2a Compare March 12, 2021 12:28
@kittaakos
Copy link
Contributor Author

I have fixed it in Theia and patched the IDE2 for the time being. Please review.

 - Reordered the sidebar views.
 - Increased the default sidebar width.

Signed-off-by: Akos Kitta <kittaakos@typefox.io>
@kittaakos kittaakos force-pushed the atl-1063--multi-tab-search branch from 58d0a2a to 6255ca4 Compare March 12, 2021 12:30
Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working great now. Thanks Akos!

@kittaakos kittaakos merged commit 5a262d4 into main Mar 15, 2021
@kittaakos kittaakos deleted the atl-1063--multi-tab-search branch March 15, 2021 08:15
@per1234 per1234 added topic: code Related to content of the project itself type: enhancement Proposed improvement labels Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants