Skip to content

Conversation

@SK-143381
Copy link
Collaborator

@SK-143381 SK-143381 commented Sep 6, 2024

This pull request addresses the following issues and implements the necessary fixes:

  1. Add Publication Page (Closes feat: add publication page #7):

    • Created a new publication page using Quarto with references sourced from a BibTeX file.
  2. Fix Issue Report Page URL Contains a Tag (Closes fix: address an issue where issue report page URI contains a tag #6):

    • Resolved the issue where the "Report an Issue" link in the footer included an erroneous <a> tag in the _quarto.yml file.
    • The link is now properly formatted, without the tag, and directs users to the correct issue reporting page on GitHub.
  3. Add Alt Text to Headshots in People Page (Closes fix(people): add alt text to headshot #5):

    • Added descriptive alt text to all headshots on the "People" page to improve accessibility.
  4. Fixed footer link colour issue as well.

SK-143381 and others added 7 commits August 26, 2024 17:24
added individual images and shifted content of the pages.
The home page now contains the lab logo in the background. The lab welcome message has been centered to the bottom of the screen. The extra sections from the people page have been commented currently. The Projects page has been updated with official funded projects.
Addressed the following issues -

1. **Add Publication Page (Closes #7)**:
   - Created a new publication page using Quarto with references sourced from a BibTeX file.
   - Publications are dynamically generated and sorted by year.
   - This addresses the feature request to create a structured, accessible, and dynamically updated publication list.

2. **Fix Issue Report Page URL Contains a Tag (Closes #6)**:
   - Resolved the issue where the "Report an Issue" link in the footer included an erroneous `<a>` tag in the `_quarto.yml` file.
   - The link is now properly formatted, without the tag, and directs users to the correct issue reporting page on GitHub.

3. **Add Alt Text to Headshots in People Page (Closes #5)**:
   - Added descriptive alt text to all headshots on the "People" page to improve accessibility.
   - Ensures that screen readers can properly interpret the images, enhancing accessibility for users with visual impairments.

4. The link colour issue has also been resolved.
Copy link
Member

@jooyoungseo jooyoungseo left a comment

Choose a reason for hiding this comment

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

LGTM! Please resolve the merge conflicts.

@SK-143381
Copy link
Collaborator Author

I have resolved merge conflicts. Please merge if you think it ready.

Copy link
Member

@jooyoungseo jooyoungseo left a comment

Choose a reason for hiding this comment

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

LGTM!

@jooyoungseo jooyoungseo merged commit c4aca04 into main Sep 6, 2024
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.

feat: add publication page fix: address an issue where issue report page URI contains a tag fix(people): add alt text to headshot

3 participants