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

flesh out and clean-up toolbar #172

Merged
merged 1 commit into from
May 28, 2024
Merged

flesh out and clean-up toolbar #172

merged 1 commit into from
May 28, 2024

Conversation

cloverich
Copy link
Owner

@cloverich cloverich commented May 28, 2024

  • Add Radix-ui dropdown menu
  • Fleshes out remaining supported block types (paragraph, heading, etc) with a toggle / insert dropdown menu
  • Remove debug tabs (evergreen component) and replace with debug dropdown in toolbar
  • Some clean-up, commenting, re-naming of toolbar and debug related functions
  • Addition of read only editor view, just a shell for now
Screenshot 2024-05-28 at 8 50 14 AM

It mostly works, but reveals a few other bugs (such as #171 ). I created a few associated Github issues to control scope.

This also removes the prior debug tabs, and converts it into a dropdown in the toolbar:
Screenshot 2024-05-28 at 8 50 23 AM

Originally I had intended to work on #128 , but realized I needed to be able to toggle regular headings, and understand some additional concepts that were related anyhow. Now I think better to flesh out all the core blocks first, before doing the more customized ones. Makes the editor more usable in a base sense anyways.

- Fleshes out remaining supported block types (paragraph, heading, etc) with a toggle / insert dropdown menu
- Remove debug tabs (evergreen component) and replace with debug dropdown in toolbar
- Some clean-up, commenting, re-naming of toolbar and debug related functions
- Addition of read only editor view, just a shell for now
@cloverich cloverich merged commit 0f2e047 into master May 28, 2024
2 checks passed
@cloverich cloverich deleted the toolbar-cleanup branch May 28, 2024 15:56
@cloverich
Copy link
Owner Author

Doh. This closes #103

@cloverich cloverich mentioned this pull request May 28, 2024
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