IceCubesApp icon indicating copy to clipboard operation
IceCubesApp copied to clipboard

Add a preview toot to the top of the display settings page.

Open xurble opened this issue 2 years ago • 8 comments

It also makes a few changes to the preview toot to accommodate them.

The (optional, default false) parse markdown flag stops the preview toot having weird highlighted regions when used as a placeholder, but allows links to look correct in settings.

xurble avatar Feb 08 '23 08:02 xurble

Nice! I was going to work on this next—a few comments about the UI. I will put them in individual comments to more easily "talk" about them if necessary.

EvilOne avatar Feb 08 '23 11:02 EvilOne

CleanShot 2023-02-08 at 05 40 34@2x

EvilOne avatar Feb 08 '23 11:02 EvilOne

CleanShot 2023-02-08 at 05 42 36@2x

One thing I notice is that because it is actionable, the state gets lost depending on what is tapped. No point adding any state to this, no matter how quick and easy it may be!

EvilOne avatar Feb 08 '23 11:02 EvilOne

CleanShot 2023-02-08 at 05 44 21@2x

The preview should stay in place. This will allow smaller phones to make changes further down the screen and still see what those changes would do.

EvilOne avatar Feb 08 '23 11:02 EvilOne

CleanShot 2023-02-08 at 05 47 08@2x

Highly recommend showing the share icon here to get rid of the "whitespace" and make it look more uniform.

EvilOne avatar Feb 08 '23 11:02 EvilOne

Again, I like what I see. I'm sorry if my suggestions don't come off as suggestions. I do mean them as such.

EvilOne avatar Feb 08 '23 11:02 EvilOne

How about a more personalised post example? I don't know what to put, but I like what Ivory did.

image

EvilOne avatar Feb 08 '23 12:02 EvilOne

Thanks for the review @EvilOne I agree with all of those. @xurble I'm fixing some issue in order to merge it for next release, because this is a very good start! We can iterate on this.

Dimillian avatar Feb 08 '23 17:02 Dimillian

I agree too, I've just been at work all day 😬

xurble avatar Feb 08 '23 18:02 xurble

No problem, I've just changed the wording and disabled the interactions which was necessary. More polish can come later :)

Dimillian avatar Feb 08 '23 18:02 Dimillian