Skip to content

Comments

Change status-message to non-flex container to avoid issues with links#398

Merged
omkar-ethz merged 1 commit intodevelopfrom
status-message-fix
Jul 31, 2025
Merged

Change status-message to non-flex container to avoid issues with links#398
omkar-ethz merged 1 commit intodevelopfrom
status-message-fix

Conversation

@omkar-ethz
Copy link
Member

Description

Without this change, if the statusMessage contains a link (an <a> element), the spaces around it are not rendered as it was inside a flex container. Fixed this by making it a non-flex container and centering text with text-align: center instead

Tests included/Docs Updated?

  • Included for each change/fix?
  • Passing? (Merge will not be approved unless this is checked)
  • Docs updated?
  • New packages used/requires npm install?

Extra Information/Screenshots

For statusMessage: "Dataset retrieval is currently down due to technical issues with Tape Library at CSCS. Visit <a href='https://scistatus.psi.ch/'>scistatus.psi.ch</a> for more information.",

Before:
Screenshot from 2025-07-31 13-01-50

After:
Screenshot from 2025-07-31 13-01-18

@omkar-ethz omkar-ethz requested a review from minottic July 31, 2025 11:02
@omkar-ethz omkar-ethz changed the title Chagne status-message to non-flex container to avoid issues with links Change status-message to non-flex container to avoid issues with links Jul 31, 2025
@omkar-ethz omkar-ethz merged commit 39f408b into develop Jul 31, 2025
7 checks passed
@omkar-ethz omkar-ethz deleted the status-message-fix branch July 31, 2025 11:32
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