Skip to content
This repository has been archived by the owner on Jun 13, 2024. It is now read-only.

Replace SelectableText with a small number of SelectableAreas #698

Open
guidezpl opened this issue May 25, 2022 · 7 comments
Open

Replace SelectableText with a small number of SelectableAreas #698

guidezpl opened this issue May 25, 2022 · 7 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers P2 Priority 2 - Should have

Comments

@guidezpl
Copy link
Member

guidezpl commented May 25, 2022

Need to check that selection works as expected with all nested apps (e.g. studies, as well as the settings screen.

See flutter/flutter#104547

@guidezpl guidezpl added enhancement New feature or request good first issue Good for newcomers P2 Priority 2 - Should have labels May 25, 2022
@AdiAr11
Copy link

AdiAr11 commented May 26, 2022

could you please explain what needs to be done here?

@guidezpl guidezpl changed the title Replace SelectableText with a small number of SelectablArea Replace SelectableText with a small number of SelectableAreas May 26, 2022
@olegzaim
Copy link

I have already started to work on that issue.

@AdiAr11
Copy link

AdiAr11 commented May 26, 2022

okay, no problem !

@roshhni97
Copy link

can I try this issue??

@guidezpl
Copy link
Member Author

@olegzaim has a PR out that fixes this issue, however 2 driver tests are failing. I'd welcome help in debugging those

@guidezpl
Copy link
Member Author

guidezpl commented Jan 12, 2023

Now closing in favor of flutter/flutter#104547

@guidezpl guidezpl closed this as not planned Won't fix, can't repro, duplicate, stale Jan 12, 2023
@guidezpl guidezpl reopened this Apr 24, 2023
@guidezpl
Copy link
Member Author

Now that flutter/flutter#104595 is closed, this issue becomes valid again in the short-/medium- term. flutter/flutter#104547 is still a long-term proposal.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers P2 Priority 2 - Should have
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants