-
Notifications
You must be signed in to change notification settings - Fork 66
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
Replace zoom slider #88
Comments
Done |
this looks great! |
I like the idea. Though I'm not sure if I'm missing some stylesheets, as after I've updated the code, the new 'slider-label' text is listed going down the page instead of horizontally (I assume). The dalliance code is automatically using the public stylesheets (http://www.biodalliance.org/release-0.13/css/dalliance-scoped.css). Are these stylesheets out of sync with the current dev version ? |
In adding the single base highlighter ruler in #111 I have added some classes which are not in the public stylesheets which would at least explain why the location text is all the way to the left and not in the centre. |
@Traksewt is right, there was a stale stylesheet in the release-0.13 directory. I try to keep the "next release" resources directory reasonably up to date, but it does sometimes slip for a week or two -- if you're using a development version of Biodalliance seriously, it's probably worth keeping your own copy of the resources (css, fonts, help) and setting the |
great thanks guys, I didn't realise the prefix option for stylesheets. The public css is looking good now, and the slider is looking great. |
Sounds like there's a need for better documentation regarding how assets get located, have opened biodalliance/dalliance-web#5 to track. |
The current we use as a zoom slider has at least two major problems:
Therefore, I suggest implementing a new slider that looks reasonably close to a standard but has feedback for toggle-zooming (maybe a greyed-out second thumb?).
The polyfill code we used to use to get sliders on Mozilla may be useful here.
The text was updated successfully, but these errors were encountered: