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

[511] UI-only: Adjust homepage banner height and fix style issues on mobile #512

Merged
merged 2 commits into from
May 28, 2024

Conversation

dlicheva
Copy link
Collaborator

@dlicheva dlicheva commented May 26, 2024

Ticket
This PR adjusts the styles for the homepage and fixes a few style issues on mobile:

  • make the home banner and Browse button smaller
  • remove white bar on the homepage on mobile
  • reduce space between the menu items on mobile
  • centre the logo on mobile
  • adjust the header style on tablet
  • change the padding on the Explorer page to match the other pages, also tweak the search label and the sub-menu.
  • align the labels on Map Analysis
  • resolve a browser warning on the Map Analysis page by moving a warning message out of the table.

Homepage banner size reduction

homepage-desktop
homepage-tablet

Mobile header adjustments

Screenshot from 2024-05-26 19-55-56

Explorer page on mobile

explorer

Map Analysis page on mobile

map-analysis

@dlicheva dlicheva linked an issue May 26, 2024 that may be closed by this pull request
@robvanderveer
Copy link
Collaborator

All magnificent

@dlicheva dlicheva changed the title Bugfix/511 UI-only: Adjust homepage banner height and fix style issues on mobile [511] UI-only: Adjust homepage banner height and fix style issues on mobile May 26, 2024
@dlicheva dlicheva merged commit 1e33416 into OWASP:main May 28, 2024
3 checks passed
@dlicheva dlicheva deleted the bugfix/511_home_fix branch May 28, 2024 13:38
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.

Remove 'browse topics' button on Home?
3 participants