Skip to content

Fix/header logo alt text#363

Open
sumitakhuli wants to merge 2 commits intoopen-source-uom:mainfrom
sumitakhuli:fix/header-logo-alt-text
Open

Fix/header logo alt text#363
sumitakhuli wants to merge 2 commits intoopen-source-uom:mainfrom
sumitakhuli:fix/header-logo-alt-text

Conversation

@sumitakhuli
Copy link
Copy Markdown
Contributor

@sumitakhuli sumitakhuli commented Feb 14, 2026

Fix #362

Updated the alt attribute of the logo Image component in Header.jsx from "Logo" (or missing) to "University of Macedonia Logo".

Related Issue

Motivation and Context

To improve accessibility for screen reader users by providing a descriptive alt text for the logo.

How Has This Been Tested?

  • Manual Verification: Verified that the code change reflects the new alt text.
  • Build: Verified that npm run build completes successfully.

Screenshots or GIF (In case of UI changes):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@GeorgeFkd
Copy link
Copy Markdown
Contributor

Alt texts and generally accessibility stuff are things we haven't had time to properly work on, if you could add alt texts to the tiles it will be usefull to the project.

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.

Header logo alt text is vague

2 participants