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

Made library more customizable #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

maripbekoff
Copy link

No description provided.

Copy link
Collaborator

@clemenceroumy clemenceroumy left a comment

Choose a reason for hiding this comment

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

Thank you for your PR and your work :)
I did a quick review on it

color: Colors.transparent,
child: SingleChildScrollView(
scrollDirection: Axis.horizontal,
child: Row(
children: widget.actions.map((type) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I like the fact to allow detaching the action bar from the TextInput, though I think it should be an option.

I prefer to keep the structure of the input like it was previously : attached to the bottom of the input. Thus, I think there are 2 possibilities :

  • The only thing needed is being able to set the action bar on the upper of the input. In this case, we should only set an option to choose where to put the bar (bottom or top, inside input).
  • You want to detach the action bar to do what you want with it. In this case, you should give more availabilities than what you did, by choosing if you want to set it under or over the input, for example.

- [x] Convert to Bold, Italic, Strikethrough
- [x] Convert to Code, Quote, Links
- [x] Convert to Heading (H1, H2, H3, H4, H5, H6) and Links
- [x] Support text direction

## Demo
![](pictures/test_edition.gif)
![](pictures/preview.gif)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you please do the same GIF as there was previously (one that show all the formatting options)

Comment on lines +4 to +5
homepage: https://github.com/maripbekoff/markdown-editable-textinput
repository: https://github.com/maripbekoff/markdown-editable-textinput
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you please re set the value as they were before :

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