Skip to content

[WorkdaySummary]: Improve current date indicator - #559

Merged
RiinaKuu merged 6 commits into
mainfrom
KEIJO-189-today-indicator
Aug 14, 2026
Merged

[WorkdaySummary]: Improve current date indicator#559
RiinaKuu merged 6 commits into
mainfrom
KEIJO-189-today-indicator

Conversation

@emilyhernandez

@emilyhernandez emilyhernandez commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

KEIJO-189

Changed current day indicator to a more subtle one.

  • date text and total hours chip are bolded
  • the "today" text will not affect the placement of the Chips in the center of the entry row.
  • the "today" text will be hidden in screen sizes below md breakpoint.

Screenshots

Screenshot 2026-08-14 at 11 52 33 Screenshot 2026-08-14 at 11 53 22

Developer's checklist

  • I wrote unit tests for relevant components

@emilyhernandez emilyhernandez changed the title Keijo 189 today indicator [WorkdaySummary]: Improve current date indicator Aug 14, 2026
@emilyhernandez emilyhernandez self-assigned this Aug 14, 2026
@emilyhernandez emilyhernandez added the improvement Improve or extend something that already exists label Aug 14, 2026
@emilyhernandez
emilyhernandez marked this pull request as ready for review August 14, 2026 10:59
RiinaKuu
RiinaKuu previously approved these changes Aug 14, 2026

@RiinaKuu RiinaKuu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My eyes are thanking 🙏 I personally liked the version where (Today) was also bolded, but it's not a deal breaker here.

@RiinaKuu
RiinaKuu merged commit 78cd866 into main Aug 14, 2026
22 of 23 checks passed
@RiinaKuu
RiinaKuu deleted the KEIJO-189-today-indicator branch August 14, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improve or extend something that already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants