Skip to content
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

Cell horizontal scrollbar is missing in FF but not in Chrome #4986

Closed
damianavila opened this issue Feb 1, 2014 · 5 comments · Fixed by #5035
Closed

Cell horizontal scrollbar is missing in FF but not in Chrome #4986

damianavila opened this issue Feb 1, 2014 · 5 comments · Fixed by #5035
Assignees
Milestone

Comments

@damianavila
Copy link
Member

Using the current master (9b8c058), in Chrome (Versión 31.0.1650.63 Ubuntu 13.04 (31.0.1650.63-0ubuntu0.13.04.1~20131204.1) we have an horizontal scrollbar when the line exceed the width of the cell:

scroll1

But if FF (26.0 - Mozilla Firefox for Ubuntu), the scrollbar is missing and the cell width grows horizontally with the content:

scroll2

I tried to investigate why the but I did not get to any solution... maybe you can help me to research on it 😉

@jdfreder
Copy link
Member

jdfreder commented Feb 5, 2014

Hi @damianavila -

I opened a PR to fix this, #5035. Tell me if that works for you.

Cheers,
Terminator 😉

@ellisonbg
Copy link
Member

@jdfreder re-opening this one :(

screen shot 2014-02-05 at 6 58 34 pm

I just realized that your fix messed up the vertical scrolling of the input area of the code cells. On FF it is adding a vertical scrollbar which somehow hides the lower border of the input area. Here is a screenshot:

@ellisonbg ellisonbg reopened this Feb 6, 2014
@jdfreder
Copy link
Member

jdfreder commented Feb 6, 2014

On FF it is adding a vertical scrollbar which somehow hides the lower border of the input area.

Is it always adding it? It should only add it when the horizontal one exists (and takes up some space).

jdfreder added a commit to jdfreder/ipython that referenced this issue Feb 6, 2014
@jdfreder
Copy link
Member

This works in master with the new flexible box model.

@damianavila
Copy link
Member Author

Great! Thanks for dealing with it... it causes a lot of problems... he he 😆

mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this issue Nov 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants