SOLVED: Links options in Theme Settings don't apply

Bricks Version: 1.5
Browser: Chrome 90
OS: macOS

Hi guys,

I’m trying to set the global style of the links in my theme settings. While it works as expected for links inside a basic text element, it doesn’t seem to apply to links inside header elements. I didn’t check all the elements, but my guess is it could affect other elements than headers…

How to reproduce:

  1. Change the styles of links in Settings → Theme Styles → Links.
  2. Add a header inside the page and insert <a href="#">test</a> in the content.

Btw, in the Links settings, there are 2 different text-decoration options: one in the typography options, and one at the bottom of the links options. This is a bit confusing… you might wanna have a look at it.

Cheers,
Maxime

Hi Maxime,
Thanks so much for your report!

What exactly do you mean by:

The Theme Styles Link Settings are currently applying to the following elements:

.brxe-accordion .accordion-content-wrapper a, .brxe-icon-box .content-wrapper a, .brxe-list a, .brxe-post-content a:not(.bricks-button), .brxe-posts .dynamic p a, .brxe-shortcode a, .brxe-tabs .tab-content a, .brxe-team-members .description a, .brxe-testimonials .testimonial-content-wrapper a, .brxe-text a, a.brxe-text, .brxe-text-basic a, a.brxe-text-basic

Best regards,
timmse

I mean this:

I can’t see .brxe-heading in that list.

Cheers,
Maxime

Aaah ok,
header and heading are two completely different things. That’s why I asked :wink:

True, .brxe-heading a is not served by the link theme styles, but hey: There is a filter for that :smiley:

Best regards,
timmse

Sorry for the confusion.

In my case, this solution doesn’t solve my issue as it’s meant to be exported in a remote template. Can’t we just add .brxke-heading to that class list?

Hi Maxime,
Consultation has revealed: The link styles inside of the theme styles are meant for text links.
Adding the “brxe-heading a” as well will most likely lead to unwanted styling for other users & existing sites.

If you need the styling to be part of a certain template, you can do this by adding a class to the link or by custom CSS.

Best regards,
timmse

What about a specific field for links in the heading options? Sorry to insist, but using custom css to customize headings links sounds overwhelming to me.

As said, we currently see no reason to adjust the link styles, nor to create a new setting for it. You are, of course, welcome to submit your suggestion on the Idea Board, so maybe we can be convinced otherwise :slight_smile: