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

Code Hints should never show horizontal scrollbar #2532

Closed
redmunds opened this issue Jan 13, 2013 · 9 comments
Closed

Code Hints should never show horizontal scrollbar #2532

redmunds opened this issue Jan 13, 2013 · 9 comments
Assignees
Milestone

Comments

@redmunds
Copy link
Contributor

I notice this when testing CSS Code Hints by @zoufahl and JS Code Hints by @iwehrman

The width of the Code Hints list should expand to fit the width of the content.

@njx
Copy link

njx commented Jan 14, 2013

Reviewed. Medium priority to @RaymondLim. We should probably have a max-width, but it should be fairly large.

@ghost ghost assigned redmunds Jan 26, 2013
@zoufahl
Copy link

zoufahl commented Jan 29, 2013

I just noticed that due to the horizontal bar it is impossible to read the last option, since the bar is infront of the text.
You can still select it.

@WebsiteDeveloper
Copy link
Contributor

it seems that changing the overflow-y property from auto to scroll fixes the behaviour.
I will submit a small pull request as soon as i get to it

@peterflynn
Copy link
Member

@WebsiteDeveloper - this bug is already assigned to the current sprint (see "Milestone" above), so @redmunds or @RaymondLim may already be working on it. Please check with them before putting too much effort into this.

@WebsiteDeveloper
Copy link
Contributor

i won't put to much effort in it because i seems fairly simple to fix but thanks for the hint

@WebsiteDeveloper
Copy link
Contributor

@RaymondLim could you please put up a comment if you are working on this otherwise i shall add a pull request for this issue

@redmunds
Copy link
Contributor Author

I started to look at this, but don't have a fix yet, so you're welcome to submit a pull request if you have a fix.

@redmunds
Copy link
Contributor Author

Note that we want to get rid of the horizontal scrollbar, so I don't think overflow-y is going to fix it.

@redmunds
Copy link
Contributor Author

Confirmed. Closing.

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

Successfully merging a pull request may close this issue.

6 participants