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

Improve navigation #13

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Improve navigation #13

wants to merge 9 commits into from

Conversation

vpietri
Copy link

@vpietri vpietri commented Dec 1, 2014

  • Hidden form on zoom popup and submit on second click
  • Add tabs on coparison view
  • Add dropdown period selector on comparison
  • Fix on css

@tsaavik
Copy link
Contributor

tsaavik commented Dec 1, 2014

1417454279
Just tested this skin+template, working great! Really like the dynazoom changes. Dunno about the replacing text with buttons, looks a little messy since it is impossible to have alignment.

P.S. if you install and don't do a full browser reload (shift-reload) you will not be able to open dynazooms and you will get the following in the javascript console:

 Uncaught Reference Error: popupGraph is not definedindex.html:1897 (anonymous function)jquery-1.10.2.min.js:5 x.event.dispatchjquery-1.10.2.min.js:5 v.handle

@vpietri
Copy link
Author

vpietri commented Dec 1, 2014

Will have a look at the alignment issue, I did'nt have the group button on this page.

I need a fresh javascript file to have the second click firering the submit :-(

@vpietri
Copy link
Author

vpietri commented Dec 1, 2014

What's the content of the "Groups on this level" button?

@tsaavik
Copy link
Contributor

tsaavik commented Dec 1, 2014

oh, my alignment compliant is in the 'home' display. Since different servers have different services its kinda jumbly. Its an aesthetics thing though, not really a bug per say.

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

Successfully merging this pull request may close these issues.

2 participants