Skip to content

Liisa/enhancement/627 site wide bugs and UI improvements#633

Open
davylis wants to merge 9 commits intodevfrom
liisa/enhancement/627-site-wide-bugs-and-ui-improvements
Open

Liisa/enhancement/627 site wide bugs and UI improvements#633
davylis wants to merge 9 commits intodevfrom
liisa/enhancement/627-site-wide-bugs-and-ui-improvements

Conversation

@davylis
Copy link
Copy Markdown
Contributor

@davylis davylis commented Mar 19, 2026

📄 Pull Request Overview

Closes #627

🔧 Changes Made

  1. Updated multiple UI components and styles to improve layout consistency and responsiveness across the site. This includes:
  • refining share button, footer rights, join-us icon hover, and news page spacing/styles
  • stabilizing z-index for the scroll-to-top button
  • improving feedback card sizing so inputs and buttons use responsive widths
  • adjusting Team Header With Mosaic spacing by replacing the old desktop spacer approach with balanced left/right padding
  • removing the chatbot toggle from the root layout
  1. Refactored presentation package section typing and data shape:
  • fixed image and sidebarLogo to structured objects with src, alt, width, and height
    -updated createSection, related types, and section data to match the new structure
  1. Improved image handling in news-related components by updating image dimensions.

  2. Cleaned up legacy feedback styling by commenting out the old feedback card.


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:

  • All needed done

@davylis davylis self-assigned this Mar 19, 2026
@davylis davylis marked this pull request as draft March 19, 2026 12:59
davylis added 8 commits March 25, 2026 10:35
Fixed viewport width and component overflow
Fixed the sharing button: Icon and text are centered. Border no longer jumps back and forth hover.
"Read More" section margin applied.
Updated the image dimensions to 1024x1024 to ensure high resolution with all screen sizes.
Updated the NewsCard image dimensions to 1024x1024.
Changed icons to be orange and added a hover effect.
Centered Team Page section relative to the page container.
Removed the ChatBotToggleButton component from the global layout
@davylis davylis changed the title Liisa/enhancement/627 site wide bugs and UI improvements [WIP] Liisa/enhancement/627 site wide bugs and UI improvements Mar 25, 2026
@davylis davylis force-pushed the liisa/enhancement/627-site-wide-bugs-and-ui-improvements branch from 7bee385 to 31d7c66 Compare March 25, 2026 08:40
Increase the component's z-index to ensure it remains on the top layer above all other page content
@davylis davylis force-pushed the liisa/enhancement/627-site-wide-bugs-and-ui-improvements branch from 83339fe to f99de3e Compare March 30, 2026 08:14
@davylis davylis changed the title [WIP] Liisa/enhancement/627 site wide bugs and UI improvements Liisa/enhancement/627 site wide bugs and UI improvements Mar 30, 2026
@davylis davylis marked this pull request as ready for review March 30, 2026 10:03
Copy link
Copy Markdown
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.

Great work! Most of the bugs have been fixed, and the overall user experience of the site is improving significantly. However, a few things still need attention:

Mobile Responsiveness: The pages are still too wide for mobile screens. You can see this when "swiping" the mobile view sideways and the page wobbles horizontally.
Image

GameArt Page: This page still crashes in the local development environment, as we discussed earlier on Discord.
Image

See also the comments below.

<Providers>
<FeedbackSideButton />
<ChatBotToggleButton />
{/* <ChatBotToggleButton /> */}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We could have deleted this, but we can leave it in for a quick rollout, just in case it's needed later.

@@ -1,80 +1,83 @@
@import url('https://fonts.googleapis.com/css2?family=Joti+One&display=swap');
//OLD FEEDBACK SCSS
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Regarding the commented-out code: we should avoid leaving unused components commented out as it adds technical debt. It's better to either keep the component active or remove it entirely. Since we have a dedicated issue coming up for cleaning up old components, let's not leave it in a commented state—could you either revert the commenting or delete the file?

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