You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2021/09/30 07:07:45 UTC

[GitHub] [cloudstack] slavkap opened a new issue #5538: UI - notification stacking on error messages

slavkap opened a new issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538


   
   ##### ISSUE TYPE
   <!-- Pick one below and delete the rest -->
   
    * Improvement Request
   
   ##### COMPONENT NAME
   <!--
   Categorize the issue, e.g. API, VR, VPN, UI, etc.
   -->
   ~~~
   UI
   ~~~
   
   ##### SUMMARY
   <!-- Explain the problem/feature briefly -->
   
   If you have multiple errors messages there is a need to clean them all one by one, which is not user-friendly. Is there an option to move the error messages a little further down from the alert and profile menus and to group or collapse the notications like this?
   
   
   ##### EXPECTED RESULTS
   Something like this
   ![image](https://user-images.githubusercontent.com/51903378/135403646-25f6e574-853e-418f-a891-f65ecc973689.png)
   
   To be able to clear all alerts
   To be able to expand all notifications
   
   ##### ACTUAL RESULTS
   ![image](https://user-images.githubusercontent.com/51903378/135402544-f1e6a5aa-8b9c-41db-b250-8a5d0270f56f.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] utchoang commented on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
utchoang commented on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-933119782


   @davidjumani I'm still figuring out where to put the clear button. I think it will be next to the transliteration tool. What do you think?
   ![image](https://user-images.githubusercontent.com/13766648/135790530-2b21d794-627b-41e8-b61b-53df0a549008.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] rohityadavcloud closed issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
rohityadavcloud closed issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] rhtyd commented on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
rhtyd commented on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-931095517


   More than stacking, I'm wondering why we've repeated errors (same message). How do you reproduce/get this @slavkap ?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] slavkap commented on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
slavkap commented on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-933154768


   @utchoang, the clear all button sounds nice. Next to the translate button is OK, or you could put the button on top of the first error msg. @davidjumani, you are right. Probably it's only me who is distracted to clean such messages while manually testing because they're hiding the buttons.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] davidjumani commented on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
davidjumani commented on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-933118964


   @slavkap The reason the failure notification persists is to get the user's attention. Success messages automatically disappear after 5 seconds. @utchoang where are you planning to add a clear all button ?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] slavkap commented on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
slavkap commented on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-931243065


   @rhtyd, this was just an example while testing (I'm not sure I could reproduce the exact error). My point is if it's possible to collapse those notifications and to move them a bit below.
   ![image](https://user-images.githubusercontent.com/51903378/135448388-6ef7a31f-5e8a-497e-960c-d27e1b032566.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] rohityadavcloud commented on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
rohityadavcloud commented on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-1002975361


   Fixed in https://github.com/apache/cloudstack/pull/5549
   @slavkap pl check latest 4.16 or the merged PR, re-open if not satisfied.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] utchoang commented on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
utchoang commented on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-933083052


   @slavkap cc @rhtyd @davidjumani Currently, ant design does not have a feature to support grouping notifications together. In my opinion, it would be more reasonable to have a button that allows users to clear all notifications. What do you think?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] utchoang edited a comment on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
utchoang edited a comment on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-933119782


   @davidjumani I'm still figuring out where to put the clear button. I think it will be next to the transliteration tool. What do you think? And it only appears when the notification is displayed.
   ![image](https://user-images.githubusercontent.com/13766648/135790530-2b21d794-627b-41e8-b61b-53df0a549008.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] utchoang edited a comment on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
utchoang edited a comment on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-933119782


   @davidjumani I'm still figuring out where to put the clear button. I think it will be next to the transliteration tool. What do you think? And it only appears when the notification is displayed. What do you think?
   ![image](https://user-images.githubusercontent.com/13766648/135790530-2b21d794-627b-41e8-b61b-53df0a549008.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] slavkap edited a comment on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
slavkap edited a comment on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-931243065


   @rhtyd, this was just an example while testing (I'm not sure I could reproduce the exact error). My point is if it's possible to collapse those notifications and to move them a bit below.
   ![image](https://user-images.githubusercontent.com/51903378/135448388-6ef7a31f-5e8a-497e-960c-d27e1b032566.png)
   
   
   I reproduced the error above:
   - expunge.interval = 20
   - destroy VM without expunging
   - try to expunge the VM after 30 sec. - the information isn't refreshed, so you can try multiple times to delete it and you will get those errors


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [cloudstack] rhtyd commented on issue #5538: UI - notification stacking on error messages

Posted by GitBox <gi...@apache.org>.
rhtyd commented on issue #5538:
URL: https://github.com/apache/cloudstack/issues/5538#issuecomment-932724068


   cc @utchoang @nvazquez @sureshanaparti 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org