Panel Heading Link Styling

[Guest]

For those that are interested in making a few changes to the base code to allow for easy configuration of content box headers that are links to specific pages.  Example, Shopping Cart box header.

 

Created Github of changes needed to be made.  Extremely Simple.

 

https://github.com/gburton/Responsive-osCommerce/pull/192/commits

[burt]

Core refused.  We do not want shopowners using this unnecessary code.

 

Show me something that you can achieve with this in place, and I will try to show you it can be achieved without it.

[Guest]

I'm not following the statement.  "Show me something that you can achieve with this in place, and I will try to show you it can be achieved without it."

 

Out of the box, the <a> in headers does not utilize the styling of .panel-heading from the <div> prior.  It utilizes the .a formating from the stylesheet.  If you want to make changes to this section you either assign a class to the <a> or change .a styling in the stylesheet which impacts the entire site. 

 

The aim is to provide style configuration of all headers without impacting the rest of the site.  Please share on how this can be done without.

[multimixer]

@@marcello

 

how about 

.panel-heading a {color:#f00}

in your user.css without changing any core code?

 

You don't need to add a specific class to each element you want to style, there are many ways to target it using the various classes already existing down the road to this element

[burt]

The aim is to provide style configuration of all headers without impacting the rest of the site.  Please share on how this can be done without.

 

You came up with the idea, so you must have something in mind for it.

 

Give an example of what you can do with this new code.

I will then attempt to show you it can be done without your new code.

[Guest]

@@marcello

 

how about 

.panel-heading a {color:#f00}

in your user.css without changing any core code?

 

You don't need to add a specific class to each element you want to style, there are many ways to target it using the various classes already existing down the road to this element

 

Nice.   I realize I can do better.

 

What are your thoughts on

 

.panel-heading>a

{

    color:inherit

}

 

This way there is one less place to have to modify the color setting

[multimixer]

@@marcello

 

What specific css rule you want to use is up to you, or, better to say, up to what exactly you want to achieve

 

using .panel-heading>a you are targeting a direct a child of a element with class .panel-heading, I'm not sure if .panel-heading has deeper a childs

 

In any case, point here, for me, is to make clear that you don't need to modify the html or any core files, just because you need to style an element somewhere

[Guest]

10-4.   Lesson learned.

 

.panel-heading a{color:inherit}