Skip to content

feat(client): added zero time to the card, list/table and map marker - #457

Open
lukaskett wants to merge 10 commits into
mainfrom
feature/events-list-distinguish
Open

feat(client): added zero time to the card, list/table and map marker#457
lukaskett wants to merge 10 commits into
mainfrom
feature/events-list-distinguish

Conversation

@lukaskett

Copy link
Copy Markdown
Contributor

Summary

Checklist

  • I linked the related issue or explained why none is needed.
  • I ran the relevant checks for this change.
  • I updated documentation or changelog entries when needed.
  • By submitting this pull request, I agree to the terms in CLA.md in the
    repository root.

… able to distinguish between two races on the same day
@lukaskett

Copy link
Copy Markdown
Contributor Author

Card

image

List/table

image

Map

image

Comment thread pnpm-workspace.yaml Outdated
Comment thread apps/client/src/pages/Home/eventListUtils.ts Outdated
@lukaskett

lukaskett commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Unify with the event detail header so that is contains also the name of the day
image

Should we extend the header with zero time as well (it's already present i the lower part, but to stay consistent with the header) ?

… in the list of events - shows: name of the day (new/added), date, zero time and location
@lukaskett

Copy link
Copy Markdown
Contributor Author

Event details

image

Card in the list of events

image

@martinkrivda

Copy link
Copy Markdown
Collaborator

I don't think we should unify the date format on the event detail page for several reasons.

First, this information is already presented on the Info tab. Additionally, the event detail page already displays the actual start times, so showing the date relative to Start 0 doesn't add much value in this context.

More importantly, we should keep the UI as clean and readable as possible by displaying only the essential information. On mobile devices, using the longer unified format means the date and location will no longer fit on a single line and will always wrap into two lines. This reduces the available vertical space for the results, which is more valuable than displaying additional date information. With the current format, shorter location names still allow both the date and location to fit on a single line: https://orienteerfeed.com/events/cmr81qokx000u07mpkkycb3fh?tab=info

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants