Skip to content
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

Context Menu, CSS Modules and React 19 #1015

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

CryZe
Copy link
Collaborator

@CryZe CryZe commented Feb 20, 2025

This replaces the context menu with our own. By doing that we no longer need to force install our npm packages, as there are no longer any conflicts. In fact, as part of this change, I updated us to the latest React 19 and even enabled the React compiler.

I also explored CSS modules for the first time, which are going to allow us to reduce conflicts between CSS rules and possibly even allow us to fully remove SASS, as the need for it is going to be reduced.

I also found out that the latest Rust nightly is broken due to their update to LLVM 20. So I pinned the version to the last working nightly version.

This replaces the context menu with our own. By doing that we no longer
need to force install our npm packages, as there are no longer any
conflicts. In fact, as part of this change, I updated us to the latest
React 19 and even enabled the React compiler.

I also explored CSS modules for the first time, which are going to allow
us to reduce conflicts between CSS rules and possibly even allow us to
fully remove SASS, as the need for it is going to be reduced.

I also found out that the latest Rust nightly is broken due to their
update to LLVM 20. So I pinned the version to the last working nightly
version.
@CryZe CryZe added enhancement A new feature or general improvement to LiveSplit One. UI The issue is about the user interface. performance Performance is suboptimal and should be improved. CI Affects the Continuous Integration. high priority This is a high priority issue. dependencies Pull requests that update a dependency file code quality The goal is to improve the quality of the code. labels Feb 20, 2025
@CryZe CryZe linked an issue Feb 20, 2025 that may be closed by this pull request
@CryZe CryZe enabled auto-merge (squash) February 20, 2025 21:38
@CryZe CryZe merged commit 349d95e into LiveSplit:master Feb 20, 2025
4 checks passed
@CryZe CryZe deleted the context-menu-and-co branch February 20, 2025 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Affects the Continuous Integration. code quality The goal is to improve the quality of the code. dependencies Pull requests that update a dependency file enhancement A new feature or general improvement to LiveSplit One. high priority This is a high priority issue. performance Performance is suboptimal and should be improved. UI The issue is about the user interface.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vendor react-contextmenu
1 participant