Dheepak Krishnamurthy

Results 442 comments of Dheepak Krishnamurthy

I haven't looked at the different backends, but in Crossterm there's different kinds of clear: ```rust pub enum ClearType { All, Purge, FromCursorDown, FromCursorUp, CurrentLine, UntilNewLine, } ``` Maybe we...

Maybe: - `RenderRef` - `RenderRefWithState` - `RenderOwned` - `RenderOwnedWithState` ? And at some point in the future we can deprecate `Widget` and `StatefulWidget`.

If we are renaming these methods, can we do something like this instead? ```rust pub trait RenderRef { type State; fn render_ref_with_state(&self, area: Rect, buf: &mut Buffer, state: Option) {...

There's a tracking issue for associated type defaults which if merged would allow for a defaults for `type State = ()`: https://github.com/rust-lang/rust/issues/29661 ```rust pub trait RenderRef { type State =...

Can we combine them into one trait? ```rust trait Render { fn render_mut(&mut self, context: &mut RenderContext) { self.render(context) } fn render(&self, context: &mut RenderContext) { } } ``` >...

I think in addition to the examples here in the repo, we could automate showing the examples on the website: - The website could show the examples for the latest...

Currently, my understand is that users open GitHub, click on the examples folder, and open whatever sounds interesting to them. The README on GitHub is below our long list of...

Another alternative is to make all single file examples also work with cargo script. A comment at the top of each example file should also do the trick.

> This leads to the history of the examples being fragmented somewhat, and extra work managing copying the updated changes over the old changes every time we release. I think...

What should the default `flex` be for the columns in a table? Currently it is `StretchLast` but I think `StretchLast` should go away. Good options are: - `Stretch` - `Start`