-
Notifications
You must be signed in to change notification settings - Fork 359
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
consultation navigation revamp #3378
Conversation
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@arpancodes was this revamping approach discussed in the EOD call or slack? |
@arpancodes team discussion tag is been used to know that, the particular UI is under discussion in our slack channel. Once, its completed, change it to need testing |
@nihal467 @arpancodes Removing the discussion tab. As discussed over slack, let's have tabs with a "More" button based on screen-size. On Mobile Screens, let's have Updates | Summary | Feed | More + as these are the most used tabs. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
👋 Hi, @arpancodes, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
Hi, This pr/issue has been automatically marked as stale because it has not had any recent activity. It will be closed if no further activity occurs for 7 more days. Thank you for your contributions. |
Hi, @gigincg, @nihal467, @khavinshankar, @mathew-alex, This pr has been automatically closed because it has not had any recent activity. Thank you for your contributions. Feel free to repopen the pr. |
Proposed Changes
fixes #2874
Demo
@coronasafe/reviewers
Merge Checklist
- [ ] Add specs that demonstrate bug / test a new feature.- [ ] Update product documentation.- [ ] Ensure that UI text is kept in I18n files.