Skip to content

Fix invalid package.json dependencies and added userSelect='none' to the Grid component of HomePage.jsx#354

Open
AggelosK004 wants to merge 1 commit intoopen-source-uom:mainfrom
AggelosK004:feature/polished-ui-selection
Open

Fix invalid package.json dependencies and added userSelect='none' to the Grid component of HomePage.jsx#354
AggelosK004 wants to merge 1 commit intoopen-source-uom:mainfrom
AggelosK004:feature/polished-ui-selection

Conversation

@AggelosK004
Copy link
Copy Markdown

@AggelosK004 AggelosK004 commented Dec 10, 2025

Description

  1. Removed duplicate entries for @chakra-ui/react, @emotion/react, and @emotion/styled in package.json. The duplicate keys were invalid JSON and prevented npm install from running successfully on a fresh clone.
  2. Added userSelect: 'none' to the Grid component in HomePage.jsx to prevent accidental text highlighting during interaction.

Related Issue

N/A (Fixes critical build error preventing local setup)

Motivation and Context

  • Build Fix:The project contained invalid JSON (duplicate dependency keys), making it impossible for new contributors to install dependencies and run the project locally.
  • UI: Clicking or dragging on the Home Page grid often resulted in ugly text selection highlighting. Disabling user select makes the component feel more like a native application.

How Has This Been Tested?

  • Build Test: verified that npm install now completes successfully without errors after removing duplicate keys.
  • UI Test: Ran the project locally (npm run dev) and verified that rapidly clicking the Grid component no longer triggers text selection.
  • Regression: Verified that the Home Page still loads and navigates correctly.

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.

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.

1 participant