Default menu for the pages that doesnt exists in menu

[nithyaasworld]

There are some pages that doesn't exist in the Navigation menu.

Examples:

1. Any member profile page other than the own profile Ex: /members/TonyJ being logged in as mess
2. Challenge details page
3. User menu --> Settings

Expected Behaviour: When such pages are accessed, we should just show the Community menu expanded with no selection indicator.

[nithyaasworld]

Tips to fix this is provided here by @ya3ya6 topcoder-platform/community-app#2663 (comment)

[luizrrodrigues]

@nithyaasworld Please let me know if this is what expected to Settings and Challenge Details page.
(Ignore menu animation/refresh for now)
Screen Recording 2019-06-25 at 20.12.18.mov.zip

[nithyaasworld]

@luizrrodrigues At 25th second, can you please remove the selection indicator that is present under the Community?

If the page is not in the menu items, lets not show any selection indicator for any of the menu items.

Thanks.

[luizrrodrigues]

@nithyaasworld You sure about that?
We showing Dashboard, My Profile and Payments in secondary menu, so I think Community need be selected.
Please confirm

[nithyaasworld]

Yes @luizrrodrigues

At 25th second of the video, you are in User menu -> Settings page. This screen is not part of the community menu (Dashboard or My Profile or Payments).

So its more appropriate if we do not display any selection indicator for such pages which are not present in the menus.

[luizrrodrigues]

@nithyaasworld So need remove indicator and secondary menu too, right?

[nithyaasworld]

Yes, right @luizrrodrigues

[nithyaasworld]

Thanks @luizrrodrigues

But the selection indicator is not shown if I click on any other first level menus being in one of the screens that doesn't exist in menu.

[luizrrodrigues]

@nithyaasworld I think we can close this one.

[SathyaJayabal]

@luizrrodrigues I dont see the fix.

The selection indicators are still not shown for some pages.

While they appear for some pages.

[luizrrodrigues]

@SathyaJayabal Okay, I see now.
We talking about arrow indicator.

[lakshmiathreya]

cc @lakshmiathreya @nithyaasworld

[luizrrodrigues]

@SathyaJayabal I thing we can pass this one, because the another issue reported here will follow up here: #62

Right?

[SathyaJayabal]

Ok, I will add this example to #62. Marking this as QA Pass, since the original issue is fixed. The selection indicator issue will be handles by #62

[SathyaJayabal]

Verified in production

[SathyaJayabal]

@bug-hunt-helper help

[SathyaJayabal]

@bug-hunt-helper help

[SathyaJayabal]

@bug-hunt-helper help

[bug-bash-helper]

Hi @SathyaJayabal.

add label

To add label to the issue, add comment like below with comma-separated list of labels:

@bug-hunt-helper add label: Label Name 1, Label Name 2, Label Name 3

remove label

To remove label from the issue, add comment like below with comma-separated list of labels:

@bug-hunt-helper remove label: Label Name 1, Label Name 2, Label Name 3