flutter-quill icon indicating copy to clipboard operation
flutter-quill copied to clipboard

Restore old functionaliy for some Quill toolbar classes

Open MacDeveloper1 opened this issue 1 year ago • 2 comments

Description

Today I have again spent time to make my project looks as it must. I do not do anything critical just restore the funcitonality of some toolbar classes as they were. You may consider this PR as a starting point for all other buttons which I didn't change. The only thing is required is change call of QuillToolbarIconButton in build methods of embed buttons, search button and maybe others. You may see what I did. I also restored old QuillToolbarSelectAlignmentButton and save it as QuillToolbarSelectTextAlignmentButton.

I need this modifications to make my project compiled and continue working. You may merge this PR or use it for understanding how to make package backward compatible. It will take less effors.

Related Issues

  • Fix #1628

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

  • [ ] Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
  • [x] No, this is not a breaking change.

MacDeveloper1 avatar Dec 20 '23 14:12 MacDeveloper1

If we solve the conflicts and merge this, does that fix your issue?

I restored only buttons which I used but there are other ones which are not fixed. I think the package should works properly for Material 2 and Material 3. This is the only thing you should care. Agree?

So if button in web looks like

image

before modifications it should looks like so after modifications but not like

image

I tested example app so you may read https://github.com/singerdmx/flutter-quill/issues/1628#issuecomment-1864783307 for results.

MacDeveloper1 avatar Dec 21 '23 09:12 MacDeveloper1

This issue can be solved without reverting, it's looks like the issue is caused by using the new Material 3 recommended widgets instead of the previous ones that work for Material 2 and 3, updating to 9.x.x require using Material 3 and the new widgets listed in Flutter 3.16 in additional to other changes.

See the following for more details and suggestions:

  • https://github.com/singerdmx/flutter-quill/issues/1628#issuecomment-2133522902

  • https://github.com/singerdmx/flutter-quill/issues/1628#issuecomment-2135463200

If you have any questions or suggestions, don't hesitate to create a new discussion or leave a comment here.

EchoEllet avatar Jun 14 '24 19:06 EchoEllet