Alexandre Bury

Results 285 comments of Alexandre Bury

Thanks for the report! For `set_weight`: > The code does not check whether i is out of bounds before use it directly. `self.children[i]` is guaranteed to panic first here, so...

In each of these cases, users could check first if `i > layout.len()` manually, and react accordingly. We _could_ add variants that return `Option` instead of panicking, but it might...

Hi, and thanks for the PR! Could [`SelectView::get_item_mut()`](https://docs.rs/cursive/latest/cursive/views/struct.SelectView.html#method.get_item_mut) solve your use-case? Or [`SelectView::iter_mut`](https://docs.rs/cursive/latest/cursive/views/struct.SelectView.html#method.iter_mut) if you need to access several items. In particular, you should be able to swap two items...

Hi, and thanks for the report! Indeed, it would be nice to unify the radio and check boxes. I'm not against adding a label to the Checkbox to match what...

Hi, and thanks for the report! Indeed, it seems the termion backend doesn't currently support shift+arrow (or ctrl+arrow). It doesn't seem to be supported [in termion itself](https://docs.rs/termion/latest/termion/event/enum.Event.html). Termion does return...

Hi, and thanks for the PR! I think I omitted a "min value" from the slider to keep it minimal: the "max value" is always the length of the slider...

Hi, and thanks for the report! We _could_ add these keys to the enum, but the main issue is going to be backend support. As far as I know, there...

Hi, and thanks for the report! I think the issue is that the message is printed at the time of the panic, before unwinding (and calling drop). Then, unwinding happens,...

Hi, and thanks for the report! > If the problem was just I should drop the result of find_name() before making another call, that would be fine with me provided...

Hi, and thanks for the report! I'm not sure I entirely understand the question. Are you looking for an application that connects to a database and shows a UI letting...