StyledObject does not implement Display about crossterm HOT 6 CLOSED

I think this will be the way to go: I will add another function so you do not need to create the DisplayableObject itself manually instead StyledObject will have this function:

pub fn into_displayable(self, screen: &'a Screen) -> DisplayableObject<'a, D>
{
     return DisplayableObject::new(screen, self)
}

So now you can call it like this:

println!("\nExample:\n\n\taws --profile {} s3 ls\n", style("test").with(Color::Yellow).into_displayable(&screen));

If you have any feedback or you don't like it please let me know. Otherwise tonight I'll release a new version.

from crossterm.

Yea It's a know problem unfortunately (if folks used the old API) allot has been changed and almost the whole API has been thrown over. It needed to be done and this crate had some crate benefits from it. If you have a few minutes I'll look into it. But am not sure there was a reason why I had removed it. I'll let you know...

from crossterm.

Oke I have been experimenting and have come up with something. I can't work further on it now need to go. But I see that this is something that needs to be implemented because it is quite useful to have this kind of functionality. Look at the code below of how I have implemented it for now:

fn main()
{
    let screen = Screen::default();
    println!("\nExample:\n\n\taws --profile {} s3 ls\n", DisplayableObject::new(&screen, &style("test").with(Color::Yellow)));
}

use std::fmt::{Formatter, Error};

pub struct DisplayableObject<'a, D:Display + 'a>
{
    styled_object: &'a StyledObject<D>,
    screen: &'a Screen,
}

impl <'a, D: Display + 'a> DisplayableObject<'a, D>
{
    pub fn new(screen: &'a Screen, styled_object: &'a StyledObject<D>) -> DisplayableObject<'a, D>
    {
        DisplayableObject { screen, styled_object }
    }
}

impl<'a, D: Display + 'a> Display for DisplayableObject<'a, D>
{
    fn fmt(&self, f: &mut Formatter) -> Result<(), Error> {
        self.styled_object.paint(&self.screen);
        return Ok(())
    }
}

This will wrap a StyledObject inside a DisplayableObject which can be used in the write functions. It works but still need to experiment with it a little further. And yea maybe if this is the way I go I'll release the version tomorrow.

And also implementing Display for StyledObject needs to be investigated so work in progresss...

You could copy the code above for now but unfortunately, you still need to update 60 rows :(

from crossterm.

That looks pretty good to me! Thanks so much for the quick implementation of this @TimonPost

from crossterm.

I published the new version with the changes. So I'll close the issue.

from crossterm.

Upgrade went pretty smooth, thanks for the quick turnaround on this.

from crossterm.