-
-
Notifications
You must be signed in to change notification settings - Fork 360
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
feat: add an impl of DoubleEndedIterator
for Columns
and Rows
(#1357)
#1358
feat: add an impl of DoubleEndedIterator
for Columns
and Rows
(#1357)
#1358
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1358 +/- ##
=====================================
Coverage 94.0% 94.1%
=====================================
Files 68 68
Lines 16360 16433 +73
=====================================
+ Hits 15393 15466 +73
Misses 967 967 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM generally.
- We should break the use of the pub fields for 0.29.0
- Some small ergonomics of the code for symmetry / better understanding
- This PR should include rows() too
b0e69c0
to
8bb01ac
Compare
Added |
DoubleEndedIterator
for Columns
(#1357)DoubleEndedIterator
for Columns
and Rows
(#1357)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (repeating for other maintainers), hold merging this closer to 0.29 - I'd guess this might be sometime in ~2-4 weeks.
8bb01ac
to
73aa535
Compare
73aa535
to
4680e6e
Compare
…atatui#1358) BREAKING-CHANGE: The `pub` modifier has been removed from fields on the `layout::rect::Columns` and `layout::rect::Rows` iterators. These fields were not intended to be public and should not have been accessed directly. Fixes: ratatui#1357
* origin/main: (21 commits) perf: implement size hints for `Rect` iterators (#1420) fix(color): fix doc test for from_hsl (#1421) docs: tweak readme (#1419) refactor(color)!: use palette types for Hsl/Hsluv conversions (#1418) chore(deps)!: pin unicode-width to 0.2.0 (#1403) feat(scrolling-regions)!: use terminal scrolling regions to stop Terminal::insert_before from flickering (#1341) feat!: add an impl of `DoubleEndedIterator` for `Columns` and `Rows` (#1358) fix(rect)!: Rect::area now returns u32 and Rect::new() no longer clamps area to u16::MAX (#1378) docs: fix missing breaking changes link (#1416) feat(line)!: impl From<Cow<str>> for Line (#1373) chore(deny): allow Zlib license in cargo-deny configuration (#1411) feat(tabs)!: allow tabs to be deselected (#1413) feat(color): add hsluv support (#1333) feat(table)!: add support for selecting column and cell (#1331) feat(text): improve concise debug view for Span,Line,Text,Style (#1410) chore: add benchmark for `Table` (#1408) refactor: Consistent result expected in layout tests (#1406) feat(logo): Add a Ratatui logo widget chore(deps): update rstest requirement from 0.22.0 to 0.23.0 (#1394) chore(deps): update octocrab requirement from 0.40.0 to 0.41.0 (#1393) ...
Closes #1357