Opened 3 years ago

Last modified 2 years 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


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.


Attachments (1)

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

Download all attachments as: .zip

Change History (10)

Changed 3 years ago by be.grooveadelic

comment:1 Changed 3 years 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 3 years 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 3 years 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 3 years 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 3 years ago by zzz

Milestone: undecided0.9.38
Priority: trivialminor
Status: newopen

comment:6 Changed 3 years ago by zzz

Parent Tickets: 2330

comment:7 Changed 3 years 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 3 years 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 2 years 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.