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

UX improvements #12

Merged
merged 6 commits into from
Nov 27, 2023
Merged

UX improvements #12

merged 6 commits into from
Nov 27, 2023

Conversation

TodePond
Copy link
Collaborator

@TodePond TodePond commented Nov 27, 2023

This PR makes some UX changes.

  • The highlighter tool is placed earlier on in the toolbar (so that it's visible - not in the overflow menu).
  • The live image tool is placed on the toolbar (so that it's visible - not in the overflow menu). Replaces the frame tool.
  • The size style defaults to XL on load, as it works better with draw fast.
  • The 'f' keyboard shortcut now selects the live image tool.
  • The 'draw fast' button now creates a frame with no prompt.
  • Frames with no prompt read 'double click prompt to edit'.
  • Fixed not being able to click on the prompt input when editing.
image image

@TodePond TodePond requested a review from steveruizok November 27, 2023 14:35
@steveruizok
Copy link

Good changes! I'd put the frame left of the draw / highlighter tool. We could also consider creating a drawing tool that is better suited to this type of use, i.e. instead of using the highlighter colors / sandwich mode, just a big fat draw tool.

@steveruizok
Copy link

Either way for this PR I think it's good to go.

@TodePond
Copy link
Collaborator Author

Cool! Moved the frame (live image) tool left :)

@TodePond TodePond merged commit 873231d into main Nov 27, 2023
@TodePond TodePond deleted the lu/toolbar branch November 27, 2023 17:11
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