You are reading a single comment by @Sir_Indy and its replies. Click here to read the full conversation.
  • This might be unpopular, but my feedback on these changes is mostly negative:

    I agree with user140377 - the checkbox is very clear on the tiny screen. Worth remembering when looking at giant icons on a computer that the real icon is ~3mm tall, and if the only change is a ~1mm tick or cross in the centre it could be hard to spot, especially if the user has bad eyesight. For the same reason, I liked the previous 'repeat' symbol. Nice and clear, obvious what it means, and symbols don't need translating. Personally I'm quite happy tapping the alarm to take me to a menu with all the options for it.

    I do like the idea to hide the Bulk Actions behind another layer though, it was making me nervous having a DELETE ALL button there that I might accidentally press!

    If you want to add more features to the alarms, directly adding a message would be nice, rather than going through Noteify.

  • I agree with user140377 - the checkbox is very clear on the tiny screen. Worth remembering when looking at giant icons on a computer that the real icon is ~3mm tall, and if the only change is a ~1mm tick or cross in the centre it could be hard to spot, especially if the user has bad eyesigh

    It was an idea to workaround the fact that we cannot directly check/uncheck an alarm AND open it because the API is quite limited now (but maybe I'm wrong).

    For the same reason, I liked the previous 'repeat' symbol. Nice and clear, obvious what it means, and symbols don't need translating.

    Imho the repeat symbol is not needed. In 07:00 MTWTF__ [X] obviosly the alarm is "repeated". The current version shows Alarm 7:00 (R) [X]: ok, the alarm is on repeat, but when?

    The problem with a repeat symbol is that it eats too many space: time + days of week + repeat icon = two lines and I'd like to keep all the info on one line.

    I could add an option so the user can choose compact view vs full view:

    (this is only a quick mock with the variants mixed up)

    (the underscores are needed to move the text to the right as spaces does not work ( @Gordon : this is the issue I was referring to in the PR))

    I do like the idea to hide the Bulk Actions behind another layer though, it was making me nervous having a DELETE ALL button there that I might accidentally press!

    All actions require a confirm ;-)

About

Avatar for Sir_Indy @Sir_Indy started