Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

UI glitches after quick open invocation #9248

Closed
larz0 opened this issue Sep 24, 2014 · 11 comments · Fixed by #9376
Closed

UI glitches after quick open invocation #9248

larz0 opened this issue Sep 24, 2014 · 11 comments · Fixed by #9376

Comments

@larz0
Copy link
Member

larz0 commented Sep 24, 2014

Sidebar selection and bottom of the sidebar get a little weird. This only happens when the app window height is short.

Before quick open:

screen shot 2014-09-23 at 9 25 39 pm

After:

screen shot 2014-09-23 at 9 25 47 pm

@redmunds
Copy link
Contributor

Seems to be yet another way to cause #3376

@RaymondLim
Copy link
Contributor

I think it's now a lot easier to hit this UI glitch. If you open Brackets src project in a smaller window size as in the screenshot and then make a multi-line selection in Brackets.js and reload Brackets, then you're sure to see this glitch. (Not reproducible with these steps in release 43 though)

Note that one of the requirements in my steps here is to have a project tree that has some folders expanded and the entire tree can be scrolled for several views.

screenshot 2014-09-24 15 19 20

@njx
Copy link

njx commented Sep 26, 2014

I'm running into this a lot with master as well.

@peterflynn
Copy link
Member

Adding 0.44 milestone since this is extremely visible -- it seems to happen 100% of the time that you use Quick Open now with the new tree.

@redmunds
Copy link
Contributor

redmunds commented Oct 1, 2014

We decided that this is too risky for 0.44, so moved to 0.45 milestone

@redmunds
Copy link
Contributor

redmunds commented Oct 1, 2014

This cannot be closed until #9203 is fixed.

@redmunds
Copy link
Contributor

redmunds commented Oct 6, 2014

@larz0 Let me know if you can still reproduce this in master.

@larz0
Copy link
Member Author

larz0 commented Oct 6, 2014

@redmunds ok cool. I haven't been able to reproduce this so far.

@njx
Copy link

njx commented Oct 6, 2014

YAY.

@redmunds
Copy link
Contributor

redmunds commented Oct 6, 2014

Yes, great job @marcelgerber tracking down the cause of this and also finding a good solution!

@larz0
Copy link
Member Author

larz0 commented Oct 7, 2014

@marcelgerber

giphy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants