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

Feat: Text Editor Buttons #25

Closed
6 tasks done
Tracked by #37
JYLN opened this issue Nov 14, 2023 · 2 comments · Fixed by #40
Closed
6 tasks done
Tracked by #37

Feat: Text Editor Buttons #25

JYLN opened this issue Nov 14, 2023 · 2 comments · Fixed by #40
Assignees
Labels
feature New feature or request

Comments

@JYLN
Copy link
Owner

JYLN commented Nov 14, 2023

Summary of Proposed Feature

Would like to include a row of text editor buttons with the notes field of time entries to easily format notes with Markdown syntax. Could potentially be worked in conjunction of #6. I'm also going to put this on the backlog because I don't find it absolutely necessary on first go around with this project.

As far as actual editor functions, I'm thinking of the following:

  • Heading
  • Bold
  • Italic
  • List (Numeric)
  • List (Unordered)
  • Link

Given the use case of this project, I'm thinking these will be the most widely used editor functions rather than tables, code blocks, etc. Will address this further upon feedback from intended users.

@JYLN JYLN added backlog Not an immediate priority feature New feature or request labels Nov 14, 2023
@JYLN JYLN self-assigned this Nov 14, 2023
@JYLN
Copy link
Owner Author

JYLN commented Nov 14, 2023

huntabyte/shadcn-svelte#488

Just today, the overall component library introduced a ToggleGroup component which could work perfectly for this case, I'm thinking adding additional styles to make it seem more like a toolbar. See above and docs page

@JYLN JYLN mentioned this issue Nov 22, 2023
2 tasks
@JYLN JYLN removed the backlog Not an immediate priority label Nov 27, 2023
@JYLN
Copy link
Owner Author

JYLN commented Nov 27, 2023

ToggleGroup may be an option in the future, but I don't wanna deal with the toggle states at the moment so that may be another rework issue in the future.

@JYLN JYLN linked a pull request Nov 29, 2023 that will close this issue
@JYLN JYLN closed this as completed in #40 Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant