Simplify ui/menu.rs
This commit is contained in:
parent
25103833b2
commit
58d08d36ae
1 changed files with 5 additions and 7 deletions
|
@ -233,16 +233,16 @@ impl<T: Item + 'static> Component for Menu<T> {
|
|||
let max_lens = self.options.iter().fold(vec![0; n], |mut acc, option| {
|
||||
let row = option.row();
|
||||
// maintain max for each column
|
||||
for (i, cell) in row.cells.iter().enumerate() {
|
||||
for (acc, cell) in acc.iter_mut().zip(row.cells.iter()) {
|
||||
let width = cell.content.width();
|
||||
if width > acc[i] {
|
||||
acc[i] = width;
|
||||
if width > *acc {
|
||||
*acc = width;
|
||||
}
|
||||
}
|
||||
|
||||
acc
|
||||
});
|
||||
let len = (max_lens.iter().sum::<usize>()) + n + 1; // +1: reserve some space for scrollbar
|
||||
let len = max_lens.iter().sum::<usize>() + n + 1; // +1: reserve some space for scrollbar
|
||||
let width = len.min(viewport.0 as usize);
|
||||
|
||||
self.widths = max_lens
|
||||
|
@ -250,9 +250,7 @@ impl<T: Item + 'static> Component for Menu<T> {
|
|||
.map(|len| Constraint::Length(len as u16))
|
||||
.collect();
|
||||
|
||||
const MAX: usize = 10;
|
||||
let height = std::cmp::min(self.options.len(), MAX);
|
||||
let height = std::cmp::min(height, viewport.1 as usize);
|
||||
let height = self.options.len().min(10).min(viewport.1 as usize);
|
||||
|
||||
self.size = (width as u16, height as u16);
|
||||
|
||||
|
|
Loading…
Add table
Reference in a new issue