Skip to content

Idrees/enhancement/475 update gallery page #513

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

Idreesyaser
Copy link
Contributor

📄 Pull Request Overview

Closes [Provide IssueNumber]

🔧 Changes Made

Updated the gallery page layout and structure:

Styled the visual appearance of gallery sections

Added placeholder content for image sets and descriptions

Implemented translations for all visible text elements

Note:
This update does not include API integrations or logic for placing images dynamically.

Also included:

Component-level testing

Checklist Before Submission

  • Functionality: I have tested my code, and it works as expected.
  • JSDoc: I have added or updated JSDoc comments for all relevant code.
  • Debugging: No console.log() or other debugging statements are left.
  • Clean Code: Removed commented-out or unnecessary code.
  • Tests: Added new tests or updated existing ones for the changes made.
  • Documentation: Documentation has been updated (if applicable).

📝 Additional Information

Provide any additional context or information that reviewers may need to know:

Näyttökuva 2025-07-30 202009

Copy link
Contributor

@Rutjake Rutjake left a comment

Choose a reason for hiding this comment

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

The page crashes when running dev, this needs to be fixed before we can continue review. Also you have installed different version of dependencies which might cause crash in my end.
image

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