Skip to content

Added Template Website for the Communities#18

Merged
SithuSoe04 merged 3 commits intomainfrom
feature/lucashlaing-OscarKhaing/FE-TemplateWebsite
Jan 30, 2025
Merged

Added Template Website for the Communities#18
SithuSoe04 merged 3 commits intomainfrom
feature/lucashlaing-OscarKhaing/FE-TemplateWebsite

Conversation

@lucashlaing
Copy link
Copy Markdown
Collaborator

Added Template Website for the Communities. Is accessible through the /templatewebsite URL.

Copy link
Copy Markdown
Collaborator

@SithuSoe04 SithuSoe04 left a comment

Choose a reason for hiding this comment

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

Great work! Looks good to me overall. A few minor fixes to be made before merge:

  1. Make the button consistent with buttons on the other pages.
  2. The Meet the Team section should not have 4 tabs since each community will be having their own page. Remove the tabs so that the user only sees the people in the appropriate teams.
  3. The left and right arrow (to go back and front) in the Meet The Team section is on the edge of the pages. Move the arrows closer to the pictures so that it feels more natural to click on the arrows.
Screenshot 2024-12-13 at 5 41 29 PM

Feel free to submit a new PR once you have make the changes!

… to the center, and matched other buttons to the same style as other page
@SithuSoe04
Copy link
Copy Markdown
Collaborator

The previous/next button looks fine when there are six people showing in the section. However, when there are only three people on the page, the buttons become misaligned. Can you please fix this?

Screenshot 2025-01-21 at 5 03 09 PM Screenshot 2025-01-21 at 5 03 30 PM

@SithuSoe04
Copy link
Copy Markdown
Collaborator

Good work!

@SithuSoe04 SithuSoe04 merged commit 3237153 into main Jan 30, 2025
@SithuSoe04 SithuSoe04 deleted the feature/lucashlaing-OscarKhaing/FE-TemplateWebsite branch January 30, 2025 21:20
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