You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Vivek Ratnavel Subramanian <vi...@gmail.com> on 2017/03/01 15:08:38 UTC
Review Request 57201: Service Alert Popup - Unexpected User Experience
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57201/
-----------------------------------------------------------
Review request for Ambari, Jaimin Jetly, Richard Zang, Xi Wang, and Yusaku Sako.
Bugs: AMBARI-20254
https://issues.apache.org/jira/browse/AMBARI-20254
Repository: ambari
Description
-------
AMBARI-19049 introduced some UI changes that causes regression in UX for the Service Alert popup.
With the change, the user has to click on the expander to see any alert context. And the expander UX is strange too. Clicking anywhere (including the hyperlinked text as well as the background) in the header takes the user to the alerts page. This is not the experience the user would expect based on conventions.
The cases in which too much context is displayed and therefore needs special handling is not the norm, so crippling UX for the special case is not the way to go.
Diffs
-----
ambari-web/app/controllers/main/service/info/summary.js b3d7b35
ambari-web/app/models/alerts/alert_definition.js 4da7199
ambari-web/app/styles/alerts.less e31ea30
ambari-web/app/templates/main/service/info/service_alert_popup.hbs 6ab6b3f
Diff: https://reviews.apache.org/r/57201/diff/1/
Testing
-------
Verified Manually.
Ambari-web unit tests pass
20571 passing (19s)
153 pending
Thanks,
Vivek Ratnavel Subramanian
Re: Review Request 57201: Service Alert Popup - Unexpected User
Experience
Posted by Yusaku Sako <yu...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57201/#review168325
-----------------------------------------------------------
Ship it!
Ship It!
- Yusaku Sako
On March 1, 2017, 7:16 p.m., Vivek Ratnavel Subramanian wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/57201/
> -----------------------------------------------------------
>
> (Updated March 1, 2017, 7:16 p.m.)
>
>
> Review request for Ambari, Jaimin Jetly, Richard Zang, Xi Wang, and Yusaku Sako.
>
>
> Bugs: AMBARI-20254
> https://issues.apache.org/jira/browse/AMBARI-20254
>
>
> Repository: ambari
>
>
> Description
> -------
>
> AMBARI-19049 introduced some UI changes that causes regression in UX for the Service Alert popup.
>
> With the change, the user has to click on the expander to see any alert context. And the expander UX is strange too. Clicking anywhere (including the hyperlinked text as well as the background) in the header takes the user to the alerts page. This is not the experience the user would expect based on conventions.
>
> The cases in which too much context is displayed and therefore needs special handling is not the norm, so crippling UX for the special case is not the way to go.
>
>
> Diffs
> -----
>
> ambari-web/app/controllers/main/service/info/summary.js b3d7b35
> ambari-web/app/models/alerts/alert_definition.js 4da7199
> ambari-web/app/styles/alerts.less e31ea30
> ambari-web/app/templates/main/service/info/service_alert_popup.hbs 6ab6b3f
>
>
> Diff: https://reviews.apache.org/r/57201/diff/2/
>
>
> Testing
> -------
>
> Verified Manually.
>
> Ambari-web unit tests pass
> 20571 passing (19s)
> 153 pending
>
>
> Thanks,
>
> Vivek Ratnavel Subramanian
>
>
Re: Review Request 57201: Service Alert Popup - Unexpected User
Experience
Posted by Vivek Ratnavel Subramanian <vi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57201/
-----------------------------------------------------------
(Updated March 1, 2017, 7:16 p.m.)
Review request for Ambari, Jaimin Jetly, Richard Zang, Xi Wang, and Yusaku Sako.
Changes
-------
Increased the height of modal popup. Verified manually.
Bugs: AMBARI-20254
https://issues.apache.org/jira/browse/AMBARI-20254
Repository: ambari
Description
-------
AMBARI-19049 introduced some UI changes that causes regression in UX for the Service Alert popup.
With the change, the user has to click on the expander to see any alert context. And the expander UX is strange too. Clicking anywhere (including the hyperlinked text as well as the background) in the header takes the user to the alerts page. This is not the experience the user would expect based on conventions.
The cases in which too much context is displayed and therefore needs special handling is not the norm, so crippling UX for the special case is not the way to go.
Diffs (updated)
-----
ambari-web/app/controllers/main/service/info/summary.js b3d7b35
ambari-web/app/models/alerts/alert_definition.js 4da7199
ambari-web/app/styles/alerts.less e31ea30
ambari-web/app/templates/main/service/info/service_alert_popup.hbs 6ab6b3f
Diff: https://reviews.apache.org/r/57201/diff/2/
Changes: https://reviews.apache.org/r/57201/diff/1-2/
Testing
-------
Verified Manually.
Ambari-web unit tests pass
20571 passing (19s)
153 pending
Thanks,
Vivek Ratnavel Subramanian