If you are extending Basic components, you got styles from there. If you
don't need those styles you will have to do one of the two things in my
opinion:

1) Exclude basic styles as we did it in MDL
2) Override some of them using theme/your css file style



2018-03-06 15:26 GMT+01:00 Carlos Rovira <carlosrov...@apache.org>:

> In fact, If I remove the styles in basic, I end with some visual changes in
> JewelExample, like SVG strokes more thick in the buttons. Don't know
> exactly why this is happening, but it's clear that styles from basic
> component should be exactly 0, and then if people wants basic styles they
> should use basic theme as a dependency.
>
> I should have a 0 styles start so Jewel styles don't be affected by any
> other inherit rule
>
> 2018-03-06 15:16 GMT+01:00 Carlos Rovira <carlosrov...@apache.org>:
>
> > Hi,
> >
> > I want to share some things that affect my work on Jewel UI set and thing
> > we should fix:
> >
> > 1.- Get rid of "style" configurations
> >
> > For example, I'm getting this output for some buttons
> >
> > <button type="button" class="TextButton" style="margin: 0px; display:
> > block;">Some</button>
> >
> > <button type="button" class="TextButton" style="margin: 10px 0px 0px;
> > display: block;">Button</button>
> >
> > I think display: block; should be part of the basic.css
> >
> > In the case of margins, I think this come from the
> >
> > <js:VerticalLayoutWithPaddingAndGap gap="10"/>
> >
> > In this case, I think would be better to affect some CSS to left the HTML
> > as clean as possible
> >
> > For me the output should be:
> >
> > <button type="button" class="TextButton">Some Label</button>
> >
> >
> > 2.- When Alex did changes to basic css and remove "*" I check that
> > textfield was behaving correctly, but I think button is getting some
> styles
> > that shouldn't be in my app
> >
> >
> > Button {
> >         border: 1px solid #808080;
> >         padding: 4px;
> >         background-color: #f8f8f8;
> >         margin: 0px;
> >         border-radius: 2px;
> > }
> >
> >
> > Button:hover {
> >         border: 1px solid #808080;
> >         padding: 4px;
> >         background-color: #e8e8e8;
> > }
> >
> >
> > Button:active {
> >         border: 1px solid #808080;
> >         padding: 4px;
> >         background-color: #d8d8d8;
> > }
> >
> > Why this styles are still there? can I remove them or put in other place?
> >
> >
> > --
> > Carlos Rovira
> > http://about.me/carlosrovira
> >
> >
>
>
> --
> Carlos Rovira
> http://about.me/carlosrovira
>



-- 

Piotr Zarzycki

Patreon: *https://www.patreon.com/piotrzarzycki
<https://www.patreon.com/piotrzarzycki>*

Reply via email to