leancodepl / leancodepl/flutter_corelibrary
TextSpans structure in leancode_markup
Open
Nobody has claimed this yet.
p: leancode_markup
- Dominant language
- Dart
- Stars
- 89
- Forks
- 12
- Avg merge
- 2d 7h
- Merged PRs (30d)
- 2
Description
Do we want to have a flat structure of text spans instead of using nested spans?
| What we have now | What I though we want to have |
|---|---|
[b]aaa [i]bbb[/i][/b] |
[b]aaa [i]bbb[/i][/b] |
TextSpan(b, aaa) + TextSpan(b+i, bbb) |
TextSpan(b, aaa + TextSpan(i, bbb)) |
Originally posted by @Albert221 in https://github.com/leancodepl/flutter_corelibrary/pull/249#discussion_r1516247806
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The issue names the leancode_markup component but no file, test, or entry point. First clarify whether TextSpan values should be flat or nested and identify the affected implementation and tests. Done means the representation choice is agreed and its expected behavior is covered by tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- dart, flutter
- Domain
- mobile-dev
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100