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

Flutter Chat UI 1.6.15 contains breaking changes from 1.6.14 #626

Open
mulderpf opened this issue Aug 15, 2024 · 3 comments
Open

Flutter Chat UI 1.6.15 contains breaking changes from 1.6.14 #626

mulderpf opened this issue Aug 15, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@mulderpf
Copy link

mulderpf commented Aug 15, 2024

General

What bug do you experience? 🐞

When I upgrade from version 1.6.14 to version 1.6.15, the messages created in all previous versions of the plug-in no longer displays the same the package is therefore not backwards compatible.

The changes in 1.6.15 don't make sense as we are now moving away from standard practice of indicating bold and italic in messaging apps (specifically, Whatsapp only requires a set of single markdown characters, not two.

How can it be reproduced? 🤔

  • Have version 1.6.14 installed and chat away with people in the app, making sure to make fancy messages with bold and italic formatting.
  • Upgrade to version 1.6.15 and look at all the messages how ugly they have become.

What behavior is expected? 💡

Messages created in previous versions of the plugin should continue to display correctly in my app. There is no way to circumvent this by doing a data migration even (not going to do that), since not everyone updates their apps at the same time.


I am trying to understand the reason for this change? I cannot see the PR which changed this to understand why this was a necessary change and I really hope that in v2, we can still display messages from v1 as they were intended. I did a quick poll and 100% of users want a single * and a single _ to indicate bold and italic as they are used to using this in my app and also in Whatsapp. I know that MD standard is different, but this is what my users have come to expect.

@mulderpf mulderpf added the bug Something isn't working label Aug 15, 2024
@demchenkoalex
Copy link
Member

this conflicts with this #440 - some people would say double is a standard, some people would say single is a standard. 1.6.15 only contains this change, feel free to keep 1.6.14. I am planning to merge v2 to main soon, so this inconsistency is only temporary. I will try to use some markdown package for v2 and if it will have configuration I will make this configurable, otherwise I will just use what that package provides 🤷

@mulderpf
Copy link
Author

mulderpf commented Aug 15, 2024

I think the problem is just consistency. This was "fixed" in 1.6.6 to only accept single wildcards and nobody really complained except one person mentioned it in the issue that was opened to "improve formatting", not change it back to what it was.

It seems to be ping pong between two different standards - I accepted that this was just the way it was in 1.6.6 and now it is changing again.

The thing is, you can never please everyone. I think once you have decided on a standard and changed the formatting to one way, it should remain forward-compatible. Or at least the major version should have changed as the library is technically not compatible with older versions (my tests are breaking).

(Sorry if this seems like I am just complaining, I am very, very grateful for this package).

@demchenkoalex
Copy link
Member

no I get what you mean and I 100% agree. It is just I hate v1 at this point cause it is so bad haha - and I am very excited about v2. So I don't want to spend time fixing v1 at this point, like at all. For quite some time I was merging everything because of that and it resulted in this mess. I will 100% make sure to either have all of this customisable or have a way to override/disable features, so everyone is happy. I am working on chat gpt / gemini example now with v2 and if everything goes smoothly I think I will have basic v2 ready very soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants