-
Notifications
You must be signed in to change notification settings - Fork 20
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
Partnersite Widget Visual Fixes #316
Comments
Reviewed |
adriancollier
added a commit
that referenced
this issue
Oct 29, 2013
[#316] Partner site widget CSS fixes
Reviewed |
I think there are still some outstanding CSS rules required for the scroll bar fixes, so re-opening this for now. |
peeb
added a commit
that referenced
this issue
Nov 6, 2013
peeb
added a commit
that referenced
this issue
Nov 7, 2013
peeb
added a commit
that referenced
this issue
Nov 7, 2013
peeb
added a commit
that referenced
this issue
Nov 7, 2013
Mac OS X does not render scrollbars by default, making this rather awkward to test on OS X. Scrollbars should be set to Always render before trying to test this on any browser on OS X. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We have 2 visual changes we need to make within the Widgets that are both bugs raised in #259
This behavious varies between browsers and systems.
The suggested solution is to ensure that the Widget always includes a Scrollbar, so that this can be included within the design to ensure that this is not a problem. We have a small side-affect that for Widgets with few projects, we have an unecessary scrollbar, but this is ok.
Suggested solution is to use a combination of reducing the font size and wrapping the text onto a lower line.
The text was updated successfully, but these errors were encountered: