Skip to content

Fix mobile horizontal container overflow in research archives#1424

Open
nathenharvey wants to merge 1 commit into
mainfrom
push-tyrzsszkootw
Open

Fix mobile horizontal container overflow in research archives#1424
nathenharvey wants to merge 1 commit into
mainfrom
push-tyrzsszkootw

Conversation

@nathenharvey
Copy link
Copy Markdown
Collaborator

@nathenharvey nathenharvey commented May 23, 2026

Updates the layout and adds a test to catch regressions.

Fixes #1422

Preview URL: https://doradotdev--pr1424-drafts-off-gtpc4slh.web.app/research

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 23, 2026

Visit the preview URL for this PR (updated for commit 1013b01):

https://doradotdev--pr1424-drafts-on-pqzwtwj1.web.app

(expires Mon, 22 Jun 2026 02:50:05 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7ad2b3cf9cceb558b493931176f998ae46924361

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces responsive styling for the research page tab links, allowing them to stack vertically and wrap text on medium-sized screens. Additionally, a new Playwright test suite was added to ensure that publication list items do not overflow their containers on mobile devices. The review feedback recommends removing debugging console.log statements from the test code to maintain clean output, as per the repository's efficiency guidelines.

Comment thread test/playwright/tests/research/index.spec.ts Outdated
Updates the layout and adds a test to catch regressions.

Fixes #1422
@nathenharvey nathenharvey marked this pull request as ready for review May 23, 2026 02:55
@nathenharvey nathenharvey requested a review from davidstanke May 23, 2026 02:55
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.

Wrapping issue on /research in mobile

1 participant