Skip to content

fix: prevent sponsors link from being clipped on mobile navbar#3086

Merged
sydseter merged 4 commits into
OWASP:masterfrom
Adarshkumar0509:fix/mobile-navbar-sponsors-clipped
Jun 8, 2026
Merged

fix: prevent sponsors link from being clipped on mobile navbar#3086
sydseter merged 4 commits into
OWASP:masterfrom
Adarshkumar0509:fix/mobile-navbar-sponsors-clipped

Conversation

@Adarshkumar0509

@Adarshkumar0509 Adarshkumar0509 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Description

fixed issue: #3085

On mobile, the navbar clips the "Sponsors" link due to whitespace-nowrap on the flex container.

Solution

  • Replaced whitespace-nowrap with flex-wrap on the links container div in app.html.heex.
  • Reduced nav link font size and padding further to fit all links in a single line on 360px screens (Samsung S8)
  • Stretched the nav container to full width so links spread evenly across the navbar on all mobile viewports

Change

  • copi.owasp.org/lib/copi_web/components/layouts/app.html.heex

screenshot

image

AI Tool Disclosure

  • My contribution does not include any AI-generated content
  • My contribution includes AI-generated content, as disclosed below:
    • AI Tools: [e.g. GitHub CoPilot, ChatGPT, JetBrains Junie etc.]
    • LLMs and versions: [e.g. GPT-4.1, Claude Haiku 4.5, Gemini 2.5 Pro etc.]
    • Prompts: [Summarize the key prompts or instructions given to the AI tools]

Affirmation

@Adarshkumar0509 Adarshkumar0509 marked this pull request as draft June 8, 2026 14:52
@Adarshkumar0509 Adarshkumar0509 marked this pull request as ready for review June 8, 2026 16:15
@Adarshkumar0509

Copy link
Copy Markdown
Contributor Author

hii @sydseter , please take a look at this , i have verified all changes locally it was working fine.

@sydseter

sydseter commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

Switch the divs that use "sm:block" so that they use "min-[730px]:block" instead. That way, the logos will disappear when they have to disappear.

@sydseter

sydseter commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

Revert the other changes.

@sydseter

sydseter commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

These are the only two lines you need to change:


@sydseter sydseter merged commit 67f6985 into OWASP:master Jun 8, 2026
9 checks passed
@sydseter

sydseter commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

@Adarshkumar0509 thank you so much!

@Adarshkumar0509 Adarshkumar0509 deleted the fix/mobile-navbar-sponsors-clipped branch June 10, 2026 13:54
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