Labels Milestones

Sample button visual enhancement #104

Merged

trentlarson merged 3 commits from button-visual-enhancement into master 1 year ago

Conversation 2 Commits 3 Files Changed 25

jose commented 1 year ago

Owner

Based on user feedback about buttons not being apparent as interactive elements. I added some visual styles to make it look more like a tactile, clickable button. For now, the changes are only applied to the Account View, but I can work on getting it everywhere needed.

Based on user feedback about buttons not being apparent as interactive elements. I added some visual styles to make it look more like a tactile, clickable button. For now, the changes are only applied to the Account View, but I can work on getting it everywhere needed.

jose added 1 commit 1 year ago

a5c3600673 Sample button visual enhancement

trentlarson commented 1 year ago

Owner

Those look very nice!

I wouldn't mind if you put those on every button.

Anyway, this is fine to merge. (If you don't do it or say anything then I'll do it later.)

Those look very nice! I wouldn't mind if you put those on every button. Anyway, this is fine to merge. (If you don't do it or say anything then I'll do it later.)

trentlarson commented 1 year ago

Owner

Should these be packaged into shared CSS code or a shared button component? (I don't mind pasting it every time... just a thought.)

Should these be packaged into shared CSS code or a shared button component? (I don't mind pasting it every time... just a thought.)

jose added 1 commit 1 year ago

3d1c46aef8 Merge branch 'master' into button-visual-enhancement

jose added 1 commit 1 year ago

b6e344a15e Propagated button improvements across views

trentlarson merged commit 3bfd54362e into master 1 year ago

trentlarson referenced this issue from a commit 1 year ago

Merge pull request 'Sample button visual enhancement' (#104) from button-visual-enhancement into master

The pull request has been merged as 3bfd54362e.

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels

No items

No Label

Milestone

Set milestone

Clear milestone

No items

No Milestone

Assignees

Assign users

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview

Loading…

Cancel

Save

There is no content yet.