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

Drawer styling bugs #1329

Closed
nickdunn opened this issue May 18, 2012 · 6 comments
Closed

Drawer styling bugs #1329

nickdunn opened this issue May 18, 2012 · 6 comments
Assignees
Milestone

Comments

@nickdunn
Copy link
Contributor

I've discovered two drawer styling bugs while migrating Documenter to use it. Both are caused by the content overflow.

Firstly if the content is long and a vertical scrollbar is created inside the drawer, the content will overflow horizontally. This is because the drawer contents has a fixed width (260) and padding (2x20) but the scrollbar encroaches into this. The result is a horizontal scrollbar:

One solution might be to add another container into the markup to take the padding.

The second bug is height. If the original Symphony page is long and requires scrolling, the drawer's height will only be the height of the visible viewport and not the height of the page:

This is caused by bottom: 0; on the drawer. Removing this and it works (in Chrome) when the page is long. But then the drawer is too short when the page doesn't scroll. Gah!

@ghost ghost assigned klaftertief May 18, 2012
@klaftertief
Copy link
Contributor

I'll have a look at it later today. Hope I have some CSS mojo left then.

@nickdunn
Copy link
Contributor Author

Cheers :-)

klaftertief added a commit to klaftertief/symphony-2 that referenced this issue May 18, 2012
@klaftertief
Copy link
Contributor

I removed the set width in the CSS. It was there only to prvent a reflow of the contents during animation.

@nilshoerrmann Wanted to let you know about the changes because I think you have the best overview about the backend styles.

@nilshoerrmann
Copy link
Contributor

Thanks, Jonas.

@nickdunn
Copy link
Contributor Author

The lack of width is pretty messy when the copy is long — you see it compress and resize as the drawer expands and contracts. I remember testing with a form a while ago and the floated form elements jumped about. Perhaps a min-width, of 300px minus whatever the maximum likely scrollbar would be (20px IE/Win), so 280 or 275? That might mitigate it somewhat.

@klaftertief
Copy link
Contributor

I thought about changing from animating the width of the vertical drawers to animate their position. I'll have a play with it tomorrow.

@brendo brendo closed this as completed in a9c5640 May 21, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants