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

Language selector behaviour is buggy on different devices #93

Closed
lastpotion opened this issue Jan 5, 2017 · 9 comments · Fixed by #143
Closed

Language selector behaviour is buggy on different devices #93

lastpotion opened this issue Jan 5, 2017 · 9 comments · Fixed by #143
Assignees

Comments

@lastpotion
Copy link

🎩 User Story

I think Language selector behaviour it's a bit strange, It might be because the selector is an ul > a element.

On Desktop, as long the user hovers the ul > a and then click on a different language its fine, but if the user clicks on the language selector itself, the click event on the a takes precedence and refreshes the page with the current language.

As per mobile, on the browser (Chrome) emulating different smartphones/tablets seems showing but the user is not able to change the language. Also I tested the app on a couple of physical smartphones (Iphones 6 and OnePlus 3) and couldn't display the language dropdown on decidim-design.

📋 Related documentation

To reproduce the error on the current build of decidim-design, just change href on the the language links to something random like /es/e or /ca/c and then try to navigate.

http://decidim-design.herokuapp.com/

🎯 Acceptance criteria

Click or tap on the language selector should not redirect to the current language, only display the dropdown.

📌 Related issues

None

@lastpotion lastpotion changed the title Language selector behaviour a bit strange. Language selector behaviour is a bit strange. Jan 9, 2017
@josepjaume
Copy link
Collaborator

@htmlboy could you check this when you have some spare time? Thanks!

@josepjaume josepjaume added the bug label Jan 19, 2017
@josepjaume josepjaume changed the title Language selector behaviour is a bit strange. Language selector behaviour is buggy on different devices Jan 19, 2017
@htmlboy
Copy link
Collaborator

htmlboy commented Jan 19, 2017

@josepjaume ok!

@htmlboy
Copy link
Collaborator

htmlboy commented Jan 27, 2017

I'm not a big fan of this Foundation dropdown because it's quite dependant on markup and doesn't block the trigger's link loading (or it doesn't block a second click, not sure about that). Before considering rewriting it, would removing the href attribute on the trigger link (the selected language) solve the bug?

@josepjaume
Copy link
Collaborator

@lastpotion please chime in.

@lastpotion
Copy link
Author

lastpotion commented Feb 1, 2017

I checked all the respective CSS and HTML possible classes/tags as for example all the z-index, etc. Also I tried removing the href attribute on the trigger link, as @htmlboy suggest, but got the same result, it's not loading the respective locale.

I reproduced the issue at staging and design deployed apps and of course on localhost, sadly any of them worked or give me any hint about what is happening.

Although, I noticed testing on my smartphone that at the same time I tried to tap on the dropdown menu item, for a tiny fraction of time the element bellow where the dropdown was it's highlighted ( as if it was selected/clicked ).

I can't even guess where this tricky behaviour is coming, so can you try to reproduce it and share your thoughts?

@josepjaume
Copy link
Collaborator

@htmlboy it looks like we nailed down the issue to a Foundation bug specific on Chrome. Related to: #953 - maybe @mrcasals can give you more details.

@htmlboy
Copy link
Collaborator

htmlboy commented Feb 15, 2017

@josepjaume do you mean it's not fixed in #143 ?

@mrcasals
Copy link
Collaborator

@htmlboy @josepjaume that PR fixes the bug for me (my bet is that @josepjaume didn't see the PR) :)

@josepjaume
Copy link
Collaborator

Oh my, I didn't see it, sorry! Good work!

@ghost ghost removed the in progress label Feb 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants