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

Cleanup #603

Open
3 of 14 tasks
woelper opened this issue Jan 8, 2025 · 2 comments
Open
3 of 14 tasks

Cleanup #603

woelper opened this issue Jan 8, 2025 · 2 comments

Comments

@woelper
Copy link
Owner

woelper commented Jan 8, 2025

This is a list of ideas for cleaning up the application

  • Make ui.rs a module folder and break out components
  • Evaluate using a trait or struct for image operators
  • Move the message sender into the Player and remove the need to pass it to load()
  • Make paint a normal image operator
  • Use egui input for shortcuts
  • Add lints, such as forced documentation
  • Add clippy lint to CI
  • Move channel display to a shader
  • Evaluate using system fonts for Simplified Chinese, Japanese, Korean and Arabic
  • Send egui context to threads and functions that need to repaint instead of relying on the app entering the next frame.
  • Work around the fact that notan egui does not calculate the rect that is not covered by panels (calculate the size by storing a rect that starts from info panel to edit panel). This rect should be used to center elements and act as a clip rect for operations such as annotate, measure and operators like crop.
  • Make sure image is premultiplied when image stack is processed
  • Replace webp-sys with webp from image
    • Make sure webp animations work
@Stoppedpuma Stoppedpuma added this to the 0.9.3 milestone Jan 8, 2025
@Stoppedpuma
Copy link
Collaborator

Keeping names consistent should be here as well! For example, things like "caret" are still left over in several parts (including my own parts like icons) and should be "chevron" going forwards.

@Stoppedpuma
Copy link
Collaborator

Marked breaking out comps as finished since this is done(?) Not sure if you had other plans or not regarding that so uncheck it if you did.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Not started
Development

No branches or pull requests

2 participants