Opened 9 months ago

Last modified 4 months ago

#2302 open enhancement

Proposal: remove shutdown icon from reload button

Reported by: be.grooveadelic Owned by: zzz
Priority: minor Milestone: 0.9.38
Component: apps/console Version: 0.9.36
Keywords: Cc: sadie, alex_wykoff, alex_the_designer
Parent Tickets: #2330

Description

I initially thought it might be a bug with the CSS. But it seems to have been a deliberate choice. Looks like a glitch visually because of that I've removed it.

Subtickets (add)

Attachments (1)

remove-shutdown-icon-for-restart-hover.diff (2.2 KB) - added by be.grooveadelic 9 months ago.

Download all attachments as: .zip

Change History (9)

Changed 9 months ago by be.grooveadelic

comment:1 Changed 9 months ago by Reportage

No, it's not a glitch. It's there to reinforce the function of the button which is to restart the router ie power off and then restart, as a response to feedback that suggested it might be mistaken for a refresh console button. If you miss the tooltip hint, you won't miss the visual indication of function!

comment:2 Changed 9 months ago by slumlord

I didn't find that to be the case at all. I found it confusing to have the Power and Restart symbols, I did not understand it to mean that it was for a "power off then restart". Having text on the buttons makes the most sense to me as that would be easily translatable, I don't think it is a good idea to assume that these symbols and their meanings would be easily understood everywhere.

As described in http://trac.i2p2.i2p/ticket/2272, we plan to reduce the available themes to one light and one dark - probably classic & midnight. I would propose removing the icons for text, as is used in the 'dark' theme.

comment:3 Changed 9 months ago by zzz

  • Cc sadie alex_wykoff alex_the_designer added

Yeah, the changes last year removed the text, and we did have issues in testing with people thinking it was a refresh button. I was fine with the fix to add the power icon on hover, but it is a little bit odd, and there may be opportunity to improve further, or perhaps we should go back to text. Adding some UI team to the CC for discussion.

comment:4 Changed 9 months ago by sadie

I agree with Slumlord. The stop/restart icons are confusing especially. We may have more feedback to either support this or not when our usability testing results are presented in the next few weeks.

comment:5 Changed 8 months ago by zzz

  • Milestone changed from undecided to 0.9.38
  • Priority changed from trivial to minor
  • Status changed from new to open

comment:6 Changed 8 months ago by zzz

  • Parent Tickets set to 2330

comment:7 Changed 6 months ago by sadie

As we are updating our iconography with .38 , these icons will be updated as well. Removing the shut down button from the restart button is preferred.

comment:8 Changed 4 months ago by sadie

if we do not land updated icons for .39, I suggest that we remove the shutdown icon from the restart button.

Note: See TracTickets for help on using tickets.