-
Notifications
You must be signed in to change notification settings - Fork 580
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
Notification Troubleshooting via nested tabbed content #7529
base: master
Are you sure you want to change the base?
Conversation
Newest code from mattermost has been published to preview environment for Git SHA 5b16ce2 |
…ttermost/docs into troubleshooting-notifications
Newest code from mattermost has been published to preview environment for Git SHA ae7317e |
|
||
.. tab:: Mobile | ||
|
||
Ensure that your Mattermost mobile preferences have notifications enabled. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't these belong in the section above Check your Mattermost preferences
? I think these are not "client settings" but Mattermost preferences when the client is Mobile.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abhijit-singh - I asked myself the very same question and decided to simplify the instructions and story we're trying to tell with this structure. Very open to further discussion with the UX on this if you feel that this approach will introduce confusion.
|
||
If you prefer to use Mattermost in a web browser, you must grant notification permissions for Mattermost in the web browser. If you don't, the web browser can block you from receiving Mattermost notifications. Select the **Chrome**, **Edge**, **Firefox**, or **Safari** tab based on your preferred web browser: | ||
|
||
.. tab:: Chrome |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but I'll need CSS assistance to achieve that outcome.
|
||
Also ensure that macOS’ **Focus mode** is turned off. | ||
|
||
.. tab:: Windows |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably make 'Windows' the first tab since I think that's the OS a majority of our users would be on.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right now, everything is alphabetically ordered. Open to reordering based on popularity.
Outstanding: