Getting rid of black widget bounds about conrod HOT 13 CLOSED

Yeah I was pondering something similar to this, I like this idea :-) perhaps instead of just passing in a Color, we could pass in an enum

pub enum Coloring {
    UseColor(f32, f32, f32),
    Reactive(Color), // Will create an "inverted" Color from a given background Color so it stands out
    Default, // A default Color will be used
}

from conrod.

Or something like this:

pub enum Coloring {
    UseColor([f32, ..4]),
    Reactive([f32, ..4]), // Will create an "inverted" Color from a given background Color so it stands out
    DefaultFromTheme(Theme), // A default Color will be used from theme
}

from conrod.

Do you think we should use the Color struct (defined in color.rs) or slices instead? We could probably change over most of the stuff in the color.rs module to work with slices if it's preferred?

from conrod.

I would prefer [f32, ..4] whenever possible. We could change stuff in color.rs and declare static named colors.

from conrod.

Yeah I like this idea :-) maybe open an issue for it?

from conrod.

Oh also, in regards to the black frame - You can get rid of it by passing 0f64 into the border argument (the number of pixels wide the frame should be) for whatever the widget is. Maybe try changing it in the example and see how you like it?

from conrod.

I think the border should be there for editing.

from conrod.

Perhaps rather than just passing a f64 for the pixel size, we pass a

pub enum UseBorder {
    Border(f64, [f32, ..4]).
    NoBorder,
}

So that user can specify color and width if they wish

from conrod.

Maybe "margin" is a better name?

pub enum UseMargin {
    Margin(0f64, Coloring).
    NoMargin,
}

from conrod.

How about "frame"? As it's kind of like a picture frame, and is also 1 less character haha.

pub enum UseFrame {
    Frame(0f64, Coloring),
    NoFrame,
}

from conrod.

I think "margin" is more familiar to people, coming from CSS.

from conrod.

As discussed on IRC, Frame -> Framing is better than Margin -> Margining.

from conrod.

Implemented in #68

from conrod.