Derive empty + wrap

[pronebird]

Hi,

Is it possible to use empty + wrap helpers at the same time? This does not seem to compile:

#[derive(tabled::Tabled)]
pub struct Model {
    #[tabled(
        rename = "Description",
        display = "tabled::derive::display::empty",
        display("tabled::derive::display::wrap", 15)
    )]
    pub description: Option<String>,
}

Reports:

#[derive(tabled::Tabled)]
    |          ^^^^^^^^^^^^^^ the trait `std::fmt::Display` is not implemented for `Option<std::string::String>

[zhiburt]

Hi @pronebird

Yet it is not :(
There's nothing stopping to implement such a option.
I am not sure why it was not considered previously.
I remember that I had thoughts about it but yet did not follow it for some reason.
Maybe I've considered such list of 'decorators' to be too complex to use.

A few adjustments here would need to be done and it would work:

tabled/tabled_derive/src/attributes/field_attr.rs

Line 16 in d0b8a34

pub display_with: Option<String>,

tabled/tabled_derive/src/attributes/field_attr.rs

Line 75 in d0b8a34

self.display_with = Some(path.value());

But yet only 1 function will be called, the last one, the first one will be ignored.
So you would need to use format or create one more function which calls the functions you want.

use tabled::Table;

#[derive(tabled::Tabled)]
pub struct Model {
    #[tabled(display("wrap_option", 15))]
    pub description: Option<String>,
    #[tabled(format("{}", tabled::derive::display::wrap(&tabled::derive::display::option(&self.description1, ""), 15)))]
    pub description1: Option<String>,
}

fn wrap_option(opt: &Option<String>, width: usize) -> String {
    tabled::derive::display::wrap(&tabled::derive::display::option(opt, ""), width)
}

fn main() {
    let data = vec![Model {
        description: Some(String::from("///////////////////////////////")),
        description1: Some(String::from(
            "///x/x/x/x/x/x/x/x/xxxxxx////////////////////",
        )),
    }];

    let table = Table::new(data);

    println!("{table}");
}

Do you think we shall support list of display and format calls?