Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UX improvements #124

Merged
merged 33 commits into from
May 1, 2024
Merged

UX improvements #124

merged 33 commits into from
May 1, 2024

Conversation

lukarenko
Copy link
Collaborator

@lukarenko lukarenko commented Apr 30, 2024

Description

arrivals-departures:

  • distance/duration column with numerical sort
  • search now works
  • export moved below table

logs:

  • search text also in to and from moorage name
  • cards: center label:value pairs

stays-moorage:

  • search filter
  • sortable duration column
  • by default sort from newest to oldest
  • added stayed_at to CSV export (converted from id)
  • remove export GPX/GeoJSON as not useful for single moorage

moorages:

  • move days to column name for total stays
  • default sort by total stay

stays:

  • fix arrived/departed column names

monitoring-history:

  • remove misleading filter title

monitoring-explore:

  • remove title
  • clearable search field
  • remove Reset button (did nothing)

units:

  • miles -> NM
  • knots -> kts
  • decimals: 1 for days, NM and kts, 2 for hours

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Improvement/refactoring (non-breaking change that doesn't add any feature but make things better)

- hide search as it does not work when filter for specific moorage is applied
- distance/duration columns with num sort
- move sortNum() to utils for reuse
- search UX: only support date filter for now
- sortable Duration column
also remove duration_unit from i18n as not used
Does not make sense, as this page displays all stays at one single moorage (one location), does not make sense to export GeoJSON.
- was missing in data (just ID)
@lukarenko lukarenko requested a review from xbgmsharp April 30, 2024 19:21
@xbgmsharp xbgmsharp merged commit 881ae67 into live May 1, 2024
3 checks passed
@xbgmsharp
Copy link
Owner

Awesome thanks

@xbgmsharp xbgmsharp deleted the search-ux branch May 1, 2024 13:10
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