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

Bottom border of Quick Open text field has wrong color #7307

Closed
peterflynn opened this issue Mar 24, 2014 · 5 comments
Closed

Bottom border of Quick Open text field has wrong color #7307

peterflynn opened this issue Mar 24, 2014 · 5 comments
Assignees
Milestone

Comments

@peterflynn
Copy link
Member

  1. Type a non-existent filename into Quick Open so there are no results
  2. Type a Goto Line query (something starting with ":")

Results:
1 - Quick Open field has a red border, except bottom border is gray
2 - Quick Open field has a blue border, except bottom border is gray

This broke due to PR #5921, so assigning to @larz0

The problem is that the "smart autocomplete" dropdown is still visible, just empty, so it collapses to just a px or so tall. That will be fixed by PR #7227, but I don't think that's going to fit in the current sprint so in the meantime our CSS will have to work around the issue.

@peterflynn peterflynn added this to the Release #38 milestone Mar 24, 2014
@larz0
Copy link
Member

larz0 commented Mar 24, 2014

This is a duplicate of #5029. I was going to fix it by removing the border completely but it doesn't look as good without the border.

@peterflynn
Copy link
Member Author

@larz0 It's not a dupe in the sense that the behavior is noticeably worse on master now. Before, you could see the focus-ring color on all 4 sides, but if you resize the window you'll see a gray line break off from that. Now, you only see 3 sides of the colored border and it's visible 100% of the time to all users (no resizing required).

This is because the recent CSS changes made the focus rectangle thinner, so it no longer sticks out past the gray rectangle. I think for Sprint 38 we should just revert that one CSS change, and when my new Quick Open widget lands (hopefully Sprint 39) then you can reinstate the thinner focus ring here.

@larz0
Copy link
Member

larz0 commented Mar 24, 2014

I think they both look pretty weird with gray border. I'd like to remove the border completely for Sprint 38 because it's has less impact (that drop down border is only used for Quick Open).

screen shot 2014-03-24 at 3 26 20 pm

screen shot 2014-03-24 at 3 25 46 pm

@larz0
Copy link
Member

larz0 commented Mar 24, 2014

@peterflynn pull request is up.

@larz0
Copy link
Member

larz0 commented Mar 27, 2014

The fix is in master now, closing.

@larz0 larz0 closed this as completed Mar 27, 2014
peterflynn added a commit that referenced this issue Apr 1, 2014
Revert PR #7316 - no longer needed to fix #7307.
Tweak the PR #7358 fix to cut down the delay before the dropdown appears to
the bare minimum.
AJDBenner pushed a commit to AJDBenner/brackets that referenced this issue Apr 8, 2015
Revert PR adobe#7316 - no longer needed to fix adobe#7307.
Tweak the PR adobe#7358 fix to cut down the delay before the dropdown appears to
the bare minimum.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants