-
Notifications
You must be signed in to change notification settings - Fork 205
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
UX Design - explore ways to handle multiple notifications on integration details page #3648
Comments
PF4 has a pattern for handling multiple notifications herehttps://github.com/patternfly/patternfly/issues/654 FYI @nding-anges @mcoker |
@dongniwang @mcoker Thanks Dongni. I just have a concern about whether the Toast notification is suitable for this use case? I suppose that Toast notification is in charge of display message of global system on the overall situation. Inline notification points out an issue of a specific use case on one UI. Here, as the error message is addressed to an integration, it is better to display by Inline notification and be close to the relative item. I just made a draft solution to have a discussion with you, see image below:
If the direction is good, I will continue to design more detail situation, please feel free to brainstorming ;-) |
@nding-anges - Thanks for posting. Here are my thoughts:
|
@dongniwang Thanks for the suggestions. It's a draft design and I will continue to do an iteration design in this direction. |
@dongniwang @mcoker Hello everyone, I updated the design solution and I'd like to have a discussion with you, please find images below: Another regarding the notification message: I notice that most of these massages have the same content since a configuration required status. Is that possible to display only one notification message if the configuration required status is continued? That will solve the problem fundamentally. Thanks~ |
Each message is actually checking a different connection in the integration then returning the more generic message. Therefore, although the message is the same the underlying reason for it being flagged will be different. I have merged in an API change to the backend that actually delivers a more detailed message the references the specific differences. However, no UI picks this up yet. Maybe, that is something that could be factored in, eg. details button + slide-out panel. On the other hand, knowing that some part of the integration is stale and editing the integration would solve it is probably sufficient for the user to be told only once. Therefore, maybe at the UI-level despite receiving several notifications, a de-duplication function could be added so only one is displayed (maybe even with a number in brackets to suggest there are (n) number of these messages)? |
@nding-anges - Thanks for the update! Is it possible to move the designs into InVision? It would be easier to review and provide feedback that way. Thank you. |
@phantomjinx Thanks for explaining the context of notification. That is really helpful and now I understand very well where the problem is. Could you please give me an example of phrase that will show on the notification message after you changed API and delivered more detail? I will integrate it into the UI design. Another question: The new design can handle multiple notifications without quota, and the current strategy is to display message one by one. Do you think that will have a huge quantity of the message, like 50,100 or even more? If so, we will consider the acceptability on UI level and also from the user standpoint. Thus, I think that 'a number in brackets to suggest there are (n) number of these messages' is a good idea for a de-duplication function. |
@dongniwang I uploaded the design on Invision please have a look: https://redhat.invisionapp.com/share/J3P909MXTHQ#/332778522_Default_Status |
So this is an example of a detail message that is produced for a SYNDESIS011, ie. 'A connection associated...' message:
To reiterate previous conversations, it was felt that showing this detail was something to be avoided in preference to the more generic messages. Therefore, the placement of this detail as detail hidden behind a button. The detail property is only used currently for SYNDESIS011 & SYNDESIS012. However, that doesn't mean more messages could take advantage of it in the future. Obviously, their syntax may vary. |
@gashcrumb @phantomjinx - do you know if we would have error messages (instead of warning messages) on this integration details page? e.g. what kind of message do we display when users are not able to publish an integration (and let's say they stay on this details page when the publishing is in progress)? |
I think there's two possibilities, one is if the API request errors out.
That'd be reported via toast notification. The second case would be a
problem in the publishing process, that's reported eventually via an error
field on the integration object. I think we may not surface that error on
the detail page at all as I think about it, it used to be the field that
would drive a red x on the integration list page.
…On Tue, Nov 27, 2018, 11:51 AM Dongni (Iris) Wang ***@***.*** wrote:
@gashcrumb <https://github.com/gashcrumb> @phantomjinx
<https://github.com/phantomjinx> - do you know if we would have error
messages (instead of warning messages) on this integration details page?
e.g. what kind of message do we display when users are not able to publish
an integration (and let's say they stay on this details page when the
publishing is in progress)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3648 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAVdrIWm3jTEPJEnH8t0z4tGoNW-Z75yks5uzW17gaJpZM4W0Xsw>
.
|
@dongniwang @phantomjinx @gashcrumb
Regarding the example above, it is clear that providing more detail info on the Notification is more helpful and makes more sense to the user than a generic message. Therefore, Inline Notification has a limitation of characters and truncated policy. Maybe, it is not suitable for display more detailed information. Also, It will be annoying on the header with too much characters. I think that we could explore another design solution for that. P.S The message should be definitely re-wrote from the user stand point and make more readable.We will show an original massage directly from the backend on the UI leve for sure. |
Re the detail message in the above comment, I don't think we want to expose any of the password or sensitive information. And from a user point of view, what's the most important information from this message? I'm assuming "PostgresDB" is the most important part of the information.
And what would be the immediate action they need to perform? Wondering if we can somehow aggregate this type of information into one message (maybe try the [n] number of message method), but allow users to see more. And we list out the connection names (based on the detailed message @phantomjinx mentioned above). |
@dongniwang From the example above, we saw clearly that the message will contain two or more aspects:
The syntax may vary... Showing all these detail info will be more helpful to the user than a single message. I was wondering if we just show a type of information into one message ( with different connection names + the[n] number of massage), we will still need to display the detail somewhere on the UI and allow the user to check. So I suggest to make enhancement design or explore another design solution. |
Hi, @dongniwang @amysueg I made a new design proposal after our discussion on UX meeting last week. Considering our short-term goal for the improvement of Multiple notifications, I suppose that we could only have two status of the message list: 1) collapsed as a default status; 2) expanding status to show all message. That is good enough to meet our first goal:
So it is not necessary to have ‘a Close individual message’ and Dismiss All button as in previous design. For the reasons below:
Thus, I think only keeping these two status of display notification will both simplifier the logic of display and the user’s behavior. Here is the link: https://redhat.invisionapp.com/share/J3P909MXTHQ#/334617056_Noti-Intro_New |
This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions! |
This is a...
The problem
When there are multiple inline notifications (>5) appearing on the integration details page, the main content got pushed down below the fold.
Expected behavior
A better way to display these inline notifications on the page.
Screenshot
Reference
See comments in #3637
The text was updated successfully, but these errors were encountered: