Opened 14 months ago

Last modified 5 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
Parent Tickets: #2330 Sensitive: no

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

Attachments (1)

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

Download all attachments as: .zip

Change History (10)

Changed 14 months ago by be.grooveadelic

comment:1 Changed 14 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 14 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 14 months ago by zzz

Cc: sadie alex_wykoff Alex 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 14 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 13 months ago by zzz

Milestone: undecided0.9.38
Priority: trivialminor
Status: newopen

comment:6 Changed 13 months ago by zzz

Parent Tickets: 2330

comment:7 Changed 11 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 8 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.

comment:9 Changed 5 months ago by sadie

Having given this more thought, I think that the best option is to remove the icons and have text only. As Slumlord pointed out earlier, not all notions are easily translated when it comes to icons. Restart and refresh as icons look similar. There are also so many icons, that these seem to just get lost. Given the importance of the function, this may be better highlighted and understood as text only on the buttons and removing the icons all together.

Note: See TracTickets for help on using tickets.