No option to NOT invert logo about commercejs-nextjs-demo-store HOT 2 CLOSED

Sure. You can just remove this bit:

 ${transparent ? '' : 'invert'}

That is a ternary operator that is adding the invert class if the transparent variable is true. If you never want the invert class, you can just remove that bit. If you want it to always be inverted, you can replace it with just invert. The ${...} bit is just a way of putting some JS code in the middle of the string.

I'll close this issue for now. Thanks for the feedback and good luck with your store!

from commercejs-nextjs-demo-store.

Hey. Thanks for the answer @ScopeyNZ . Few more things. Why are anchor tags put inside next link? I've been trying to google it but can't find anything of value. Why not only the Link tag when navigating inside the app? Why the double href?

for example: components\common\Header.js

        <Link href="/collection">
          <a href="/collection" className="mr-4 font-color-black">Shop</a>
        </Link>

        <Link href="/about">
          <a href="/about" className="mr-4 font-color-black">
            About
          </a>
        </Link>

I would also like to ask about the eslint warning for target:"_blank"

This occurs in components\common\Footer.js and pages\about.js

          <a
            href="https://commercejs.com/company/about"
            className="mb-3 d-block font-color-medium"
            target="_blank"
          >
            About
          </a>

Is this something you should fix or how is this warning to be considered?

There is also an accessibility related eslint warning about the anchor tag in components\homepage\CategoryBanner.js

This doesn't worry me so much, but I wanted to point it out for you just in case you need to do something about it.

Should I open these things as a new issue or is this comment enough?

from commercejs-nextjs-demo-store.