Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

Tenacity Theme and UI/UX Changes #49

Merged
merged 9 commits into from
Aug 21, 2022
Merged

Tenacity Theme and UI/UX Changes #49

merged 9 commits into from
Aug 21, 2022

Conversation

generic-pers0n
Copy link
Member

@generic-pers0n generic-pers0n commented Aug 16, 2022

(Partially) resolves: #43 (GitHub please don't close this issue on merge)

Merges Tenacity themes into Saucedacity along with other changes.

  • I made sure the code compiles on my machine
  • I made sure there are no unnecessary changes in the code
  • I made sure the title of the PR reflects the core meaning of the issue you are solving
  • I made sure the commit message(s) contain a description and answer the question "Why do those changes fix that particular issue?" or "Why are those changes really necessary as improvements?"

* Add custom draw tool cursor
* Replace most old mouse cursors
* Increase consistency with toolbar icons
* Replace zoom icons in FreqWindow
* Increase consistency across themes
* Add custom selection adjustment cursor

(The Windows ones looked like they originated from the DOS era.)

=================================================================

Cherry-picked from Tenacity commit
b0d9f805a657f9161cf023909c54b1b97f0732aa.

Co-authored-by: Panagiotis Vasilopoulos <hello@alwayslivid.com>
Signed-off-by: akleja <storspov@gmail.com>
Signed-off-by: Panagiotis Vasilopoulos <hello@alwayslivid.com>
Signed-off-by: Avery King <avery98@pm.me>
@generic-pers0n
Copy link
Member Author

I was stripped of time to make an actual PR merging Tenacity's themes and related changes. This is why this PR is currently a draft.

Once everything is set, I will mark this as "ready to review", so we can begin the process of merging.

akleja and others added 6 commits August 19, 2022 20:43
* Colours applied to new elements from upstream
* Light theme remake
* HiContrast theme remake
* Classic Theme remake
* Removal of black border and shadows around tracks
* Decreased contrast of bevels around TrackVRulerControls to make it fit better with the new themes.
* Tweaks to track lock symbols
* The unused colors have been reverted to their old colors to prevent unforeseen consequences.

Cherry-picked from Tenacity commit f30d516. Changes have been made in
order to let Saucedacity build, such as adjusting kVerticalPadding to 6
in order to statisfy a compile-time assertion.

Signed-off-by: Avery King <avery98@pm.me>
Co-authored-by: akleja <storspov@gmail.com>
Fixes a number of small coloring mistakes.
Changes GraphLines color in light theme to black.

Signed-off-by: akleja <storspov@gmail.com>
Makes the theme respond better to system colors

Signed-off-by: akleja <storspov@gmail.com>
Adds modifier key to temporarily override sync-lock
Improves the workflow when using synced tracks
Changes Background of synced tracks
Makes locked background tile without gaps and offsets

Signed-off-by: akleja <storspov@gmail.com>
These changes are intended to bring the About and Splash dialogs more
inline with these new themes. We no longer explicitly set any colors on
any of the pages in the about dialog. We also no longer theme the Splash
dialog either (although I think it can be themed, just better than how
it was before).

Signed-off-by: Avery King <avery98@pm.me>
@generic-pers0n generic-pers0n marked this pull request as ready for review August 20, 2022 04:10
Themes in Saucedacity have been reorganized quite dramatically. The
following changes have been made:

* Tenacity's dark theme is now the default theme. It's a pretty snazzy
  theme in my opinion also.

* Renamed the "Default" theme to "Saucedacity". It will still be
  available until the end of time.

* Renamed the "Classic" theme to "Audacity Classic" to show how this
  theme is Audacity's classic theme.

Signed-off-by: Avery King <avery98@pm.me>
@generic-pers0n
Copy link
Member Author

Ok. This PR is ready to be test-driven. The full list of changes follows below:

  • Added Tenacity's themes
    • We use Tenacity's dark theme by default.
    • Our previous default theme has been renamed to "Saucedacity".
    • Renamed "Classic" to "Audacity Classic"
  • Cherry-picked Tenacity's cursor remakes. They look better than ours.

n0toose
n0toose previously approved these changes Aug 20, 2022
Copy link
Member

@n0toose n0toose left a comment

Choose a reason for hiding this comment

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

❤️

Added a little note about the inclusion of Tenacity code in the about
dialog.

Signed-off-by: Avery King <avery98@pm.me>
@generic-pers0n
Copy link
Member Author

Alright. Everything is finally done now. I should be merging pretty soon!

@generic-pers0n generic-pers0n added the enhancement New feature or request label Aug 21, 2022
@generic-pers0n generic-pers0n merged commit a1f3a62 into main Aug 21, 2022
@generic-pers0n
Copy link
Member Author

Tenacity's themes (and other changes) have been merged. Enjoy! 😄

@TheEvilSkeleton TheEvilSkeleton deleted the tenacity-themes branch August 22, 2022 14:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tenacity Merges
3 participants