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

fix: Tagline language refresh #4687

Conversation

WildOrangutan
Copy link
Contributor

What

Tagline didn't refresh, after language was changed. Convert Tagline widget to StatefulWidget.

I'm not sure if there are better ways to fix this. I'm not the most experienced Flutter developer. Please let me know what you think and I can update it accordingly otherwise.

Screenshot

untitled.webm

Fixes bug(s)

Fixes #4666

Convert Tagline widget to StatefulWIdget.
Tagline didn't refresh, after language was changed.
@codecov-commenter
Copy link

Codecov Report

Merging #4687 (f10fa99) into develop (6820165) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           develop   #4687      +/-   ##
==========================================
- Coverage     9.91%   9.90%   -0.01%     
==========================================
  Files          310     310              
  Lines        15795   15794       -1     
==========================================
- Hits          1566    1565       -1     
  Misses       14229   14229              
Files Coverage Δ
...mooth_app/lib/widgets/smooth_product_carousel.dart 0.91% <0.00%> (-0.46%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Collaborator

@g123k g123k left a comment

Choose a reason for hiding this comment

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

I would prefer an implementation where 6ou list to language changes.
This one is a "magic" fix, which is OK in terms of behavior, but not what's expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When we change the language, the tagline is not refreshed
3 participants