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

simplify language and website selection #2415

Closed
sgiehl opened this issue May 8, 2011 · 5 comments
Closed

simplify language and website selection #2415

sgiehl opened this issue May 8, 2011 · 5 comments
Assignees
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Milestone

Comments

@sgiehl
Copy link
Member

sgiehl commented May 8, 2011

the currently used fdd2div javascript is a bit crappy and the website selection quite too complicated

We should:

  • remove fdd2div and use native jquery (ui) functions instead
  • try to simplify the website selection (no different implemetations depeding on number of websites)
@sgiehl
Copy link
Member Author

sgiehl commented May 8, 2011

(In [4660]) fixes #2415

  • removed fdd2div js
  • replaced fdd2div with jquery ui autocomplete in language selection
  • simplified site selector - now using same code for both variants

@sgiehl
Copy link
Member Author

sgiehl commented May 9, 2011

(In [4661]) refs #2415 added missing doc block

@mattab
Copy link
Member

mattab commented May 10, 2011

good stuff!!
some feeedback

  • when there is one website only, the website selector should not have the arrow on the right and not be clickable
  • when search for "aiehgeaiuhg" and there is no website with such name, the line "There is no site for XXX" is clickable and leads to error, instead it should not be clickable

Note: to display search in site selector, you can set[General] autocomplete_min_sites = 2 in config file

@sgiehl
Copy link
Member Author

sgiehl commented May 10, 2011

(In [4671]) refs #2415 fixed error occuring on clicking empty-result message

@sgiehl
Copy link
Member Author

sgiehl commented May 10, 2011

(In [4672]) fixes #2415 handle case with only one site correct

@sgiehl sgiehl added this to the 1.5 - Piwik 1.5 milestone Jul 8, 2014
@sgiehl sgiehl self-assigned this Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Development

No branches or pull requests

2 participants