fossasia / open-event-frontend

The frontend for the Open Event API Server https://test.eventyay.com
https://eventyay.com
Apache License 2.0
2.34k stars 1.82k forks source link

feature 9069:Improve mobile top menu and add calendar icon #9104

Closed ntthitrinh closed 10 months ago

ntthitrinh commented 10 months ago

Fixes #9069

Short description of what this resolves:

Improve mobile top menu and add calendar icon

Changes proposed in this pull request:

-Improve mobile top menu and add calendar icon

Checklist

vercel[bot] commented 10 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
open-event-frontend ❌ Failed (Inspect) Aug 16, 2023 6:46am
mariobehling commented 10 months ago

Could you change this to something similar to the following please.

Screenshot from 2023-08-14 20-14-56

ntthitrinh commented 10 months ago

Hi @mariobehling, I think that if we make such a change, when logging in, some screens might not have enough space to accommodate all three items at the top. For instance, consider the images below: image image

mariobehling commented 10 months ago

This issue is only relevant for public event pages. It is not relevant for system or organizer pages. Therefore it should fit. There is no search bar or other things on this menu.

Screenshot from 2023-08-14 20-14-56

ghost commented 10 months ago

This issue is only relevant for public event pages. It is not relevant for system or organizer pages. Therefore it should fit. There is no search bar or other things on this menu.

Screenshot from 2023-08-14 20-14-56

Hi @mariobehling, So I just summarize that we only display 3 buttons : Speakers, Schedule, Exhibition on the public event page. If user moves to other page like: schedule, speaker, etc. That 3 buttons will removed.

Apart from that, I think it could not be shown in the mobile with phone screen size is small such as 6 inches and less. How do you think about it? please share me your idea.

ntthitrinh commented 10 months ago

Hi @mariobehling, @lthanhhieu,

image I've fixed it locally, but there isn't enough space to accommodate all the items, even though I've removed spaces between them and made the text smaller.

mariobehling commented 10 months ago

Please try to decrease the size a bit more. Thanks.

khangon commented 10 months ago

Hi @norbusan , @mariobehling , @cweitat , We have completed ticket #9069 Please help me check and verify. Thanks, Screenshot from 2023-08-15 18-42-12

khangon commented 10 months ago

Please reduce font size to allow some spacing and resolve conflicts.

Hi @mariobehling , We have fixed it. Please help me check and verify. Thanks, Screenshot from 2023-08-16 14-37-22