topcoder-platform / navigation-component

3 stars 22 forks source link

$75 | “More” tab is not swapping position with the first element to the left of the tab #101

Open codejamtc opened 5 years ago

codejamtc commented 5 years ago

https://www.screencast.com/t/Gr2VNvQ8

  1. Open the application https://test-community-app.topcoder-dev.com/challenges
  2. Login as dan_developer (Landscape)
  3. Tap Menu > Work > More > Community

Actual: “More” tab is not swapping position with the first element to the left of the tab

Expected: Selecting an option from the “More” tab should become visible and swap position with the first element to the left of the tab. This functionality can be observed on Google search.

Reproducibility Rate: 3/3

Environment: Samsung Galaxy Tab A (2017), 7 inch

Browser: Chrome 75.0.3770.80

yoution commented 4 years ago

please assign to me

yoution commented 4 years ago

how to reproduct the problem, the 'more' button is not show, how to show it

image

luizrrodrigues commented 4 years ago

@yoution Open in Firefox or reduce window size.

yoution commented 4 years ago

image

if I click 'explore','complete' go to the 'explore' position or go to the first place before 'track'? there is only three tab in my account, could you give me a test account has more tabs

luizrrodrigues commented 4 years ago

@yoution This menu is same to all accounts.

What we looking is:

So in your screenshot above will be:
Explore - More [ Tracks - Compete ]

Like as Google Search menu works, check:

Screen Recording 2020-01-09 at 22.39.38.mov.zip

yoution commented 4 years ago

https://github.com/topcoder-platform/navigation-component/pull/83

I have pushed , please review

yoution commented 4 years ago

please use this, my workflow is someting wrong, https://github.com/topcoder-platform/navigation-component/pull/89

luizrrodrigues commented 4 years ago

@yoution I see the swapping, but if click in level 3 menu will swap again to original position.

Please check:

Screen Recording 2020-01-14 at 07.35.03.mov.zip

yoution commented 4 years ago

@luizrrodrigues please review

yoution commented 4 years ago

https://github.com/topcoder-platform/navigation-component/pull/107

luizrrodrigues commented 4 years ago

@yoution If access link directly, like: http://local.topcoder.com/community/qa

swapping not working, still showing Community -> Compete - More

yoution commented 4 years ago

@luizrrodrigues https://github.com/topcoder-platform/navigation-component/pull/107 two commit , I add new commit,please review

luizrrodrigues commented 4 years ago

@yoution Looks like not working in Firefox, working well in another browsers.

yoution commented 4 years ago

@luizrrodrigues https://monosnap.com/file/waNYOBDyc4onocBc66Y1UfDlBxyoHM in my computer it work well

luizrrodrigues commented 4 years ago

@yoution I'll recheck.

luizrrodrigues commented 4 years ago

@yoution Looks good, I just redeploy from fresh repo and works well.

luizrrodrigues commented 4 years ago

@yoution please check: https://github.com/topcoder-platform/navigation-component/pull/107#issuecomment-575841552

luizrrodrigues commented 4 years ago

@yoution

Please check:

Screen Recording 2020-01-21 at 22.07.26.mov.zip

Same related in #141

This happen you go to same second level menu, ex.: 1) Select Tracks -> QA 2) Select Explore 3) Back to Tracks (see the issue)

yoution commented 4 years ago

@luizrrodrigues the green indicator not show correctly when switch tab?I will check

luizrrodrigues commented 4 years ago

Yes, Green indicator show in wrong position when follow the steps above.

yoution commented 4 years ago

@luizrrodrigues please review https://github.com/topcoder-platform/navigation-component/pull/128

luizrrodrigues commented 4 years ago

@yoution Please check video bellow, issue still exist:

Screen Recording 2020-01-27 at 20.51.36.mov.zip

yoution commented 4 years ago

@luizrrodrigues please review again, thanks https://github.com/topcoder-platform/navigation-component/pull/149

luizrrodrigues commented 4 years ago

@yoution Cool, looks good now.

SathyaJayabal commented 4 years ago

I am sorry but the issue in #141 still happens. See video

Screenshot 2020-01-28 at 6 42 49 PM

more issue.mov.zip

luizrrodrigues commented 4 years ago

@yoution This happen just showing "More" button, because function will try replace menu in More with no existing out from More.

So when screen in small and just More is showing, need ignore swipe function.

Make sense?

SathyaJayabal commented 4 years ago

@luizrrodrigues , Please make sure this issue is also fixed. On a tab size window, when you click on more->menu->sub menu, the selection indicator moves from more to Community https://github.com/topcoder-platform/navigation-component/issues/62#issuecomment-567965305

selection indicator.mov.zip

yoution commented 4 years ago

@luizrrodrigues https://github.com/topcoder-platform/navigation-component/pull/150 have pushed, please review

yoution commented 4 years ago

@luizrrodrigues I have pushed again, please review this one https://github.com/topcoder-platform/navigation-component/pull/151 . thanks