Skip to content

Resolve "Follow up & WCAG optimizations - Dashboard / Index / Topbar / Footer"

MS requested to merge 1278-follow-up-wcag-optimizations-dashboard-index into master

Closes #1278 (closed)

What does this MR do?

👋

APIS

  • Quiz → complete missing (Maybe in User Details API?)
  • Errors → complete missing (Maybe in User Details API?)
  • Broadcast → complete missing (Maybe in User Details API?) #1284 (closed)
  • Events – complete missing #1283
  • Groups – complete missing #1282
  • Regions – complete missing #1281
  • User – stat_fetchweight & stat_fetchcount are missing for the intro module
  • User – permissions missing, for the top bar handling

Reactive Datastores

  • TopBar → Uses for UnreadMailBoxCount now User-store
  • Dashboard & TopBar → All user information are now one User-store
  • Dashboard & TopBar → Stores are now from one Stores-store
  • Dashboard & TopBar → Baskets are now from one Baskets-store
  • Dashboard & TopBar → Groups are now from one Groups-store (BUT FAKED, used to get the data from top bar to dashboard)
  • Dashboard & TopBar → Regions are now from one Regions-store (BUT FAKED, used to get the data from top bar to dashboard)
  • Dashboard → Events are now from one Events-store (BUT FAKED, only storing the parameters of the backend)
  • Dashboard → Pickups getting the data from Pickups-store

Dashboard

  • Chat → LONG non-breakable words, wrapping now
  • EventField → truncate indicator is not visible
  • Avatar → alt tag added
  • IntroField → h3 to h1
  • All links → add alt
  • All actions → a with href="#" should be changed to Button
  • ActivityPost → Click to expanded should work now (bubbling was active)
  • TopBar → Icon size increased from default 14px to 16px
  • TopBar → Even more reduced, Information and Actions are now in the footer
  • TopBar → Mailbox is gone from the main menu and is moved to the Profile Menu, but with an indicator
  • TopBar → Logged out sidebar has now a direct login-form for mobile devices
  • Information → Push is hidden on Safari or iOS
  • Information → Calendar Sync is only visible when a user has no calendar token
  • TopBar → Pop out width increased, cause long chat names, long store names were missing specific information
  • TopBar → Added link to release notes and changelogs on isBeta and isDev
  • PickupList → Improved the more information to the panel, it shows now max. 4 avatars

Index

  • Moved render style from CNT_TOP to CNT_MAIN
  • Adjusted lazy loading on images
  • Added container on some content to reduce bouncing areas

Footer

  • Footer TikTok Link added (Closes #1248 (closed)), and wrong behavior for URL checks.
  • Added information and action tab links to footer
  • Column breakpoints adjusted
  • Moved the url check, for isDev, isDotAt or isBeta to frontend
  • Added urls to urls.js including Social Media

How confident are you it won't break things if deployed?

💯, on the frontend side it changes to much

Links to related issues

Screenshots (if applicable)

Screenshots

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • set a "for:" label to indicate who will be affected by this change
  • added to the next milestone (see https://gitlab.com/foodsharing-dev/foodsharing/-/milestones, unless it has a "for:Dev" label)
  • added an entry to CHANGELOG.md
  • added a short text that can be used in the release notes
  • Once your MR has been merged, you are responsible to create a testing issue in the Beta Testing forum: https://foodsharing.de/?page=bezirk&bid=734&sub=forum. Please change the MRs label to "state:Beta testing".
    • Consider writing a detailed description in German.
    • Describe in a few sentences, what should be tested from a user perspective.
    • Also mention different settings (e.g. different browsers, roles, ...) how this change can be tested.
    • Be aware, that also non technical people should understand.

Release notes text in German

Edited by MS

Merge request reports