• passepartout@feddit.org
      link
      fedilink
      arrow-up
      22
      arrow-down
      1
      ·
      6 days ago

      Yes, but element selectors based on the tag name alone are considered bad practise. Especially in bigger HTML sites. Even more so if you use a JS library / framework or some broad browser stylesheet underneath, or you use the same stylesheet in several places.

      Also, a div or img tag can be a button with its own event listener as well. That may be asking for trouble as well, but I’ve been there (not proud of it).

      • bleistift2@sopuli.xyz
        link
        fedilink
        English
        arrow-up
        6
        ·
        6 days ago

        A use case I find perfectly valid are <label class="btn">s for checkboxes and radios with huge-ass hit areas.

      • marcos@lemmy.world
        link
        fedilink
        arrow-up
        6
        arrow-down
        2
        ·
        6 days ago

        are considered bad practise.

        Considered by whom?

        The reasons that made a bunch of people scream very loudly to not do that 20 years ago were never good enough for making generic advice and don’t exist anymore.

        Even more so if you use a JS library / framework or some broad browser stylesheet underneath

        Learn CSS layers, use CSS layers.

        Overall, if you every feel the need to write in your code “a button (behaves like a button)”, it’s very likely you are doing something wrong.

        • Eager Eagle@lemmy.world
          link
          fedilink
          English
          arrow-up
          2
          ·
          edit-2
          6 days ago

          I’m not sold on layers, they do nothing to simplify the current mess that CSS is, because they build on top of what’s already out there. If CSS was built with the concept of layers, then it could probably be better. In reality, we’ll always have specificity rules and IMO adding layers will just add to the complexity.

          The specificity rule is not that hard to understand, I’m not sure what’s the point of throwing layers in the mix. There are new CSS features I like and I see simplifying written code, but this is definitely not one of them.

          • marcos@lemmy.world
            link
            fedilink
            arrow-up
            3
            ·
            6 days ago

            They change the precedence rules so that you can import large frameworks at will and not have them disrupt your rules or each other.

            Basically, anything you apply in one layer has less precedence than what is applied on the next layers.

    • bleistift2@sopuli.xyz
      link
      fedilink
      English
      arrow-up
      6
      ·
      6 days ago

      Not every button should have the usual button-y style. A close button, for instance, might be displayed differently. Or a play button layered on top of a video.

      • dohpaz42@lemmy.world
        link
        fedilink
        English
        arrow-up
        4
        ·
        6 days ago

        I imagine the better argument (in my opinion) would be to not use generic names such as btn, as they are redundant and offer little self-documentation of their use.

        Instead, I believe a combination of selectors and classes would be more meaningful, and encourage readability. For this particular scenario, I would go with a button selector to give all <button> tags a consistent basic styling, and use aptly-named classes like .cancel, .primary, and .destructive to add more styling for specific button purposes.

        Then if I need to find all Cancel buttons, I can use a button.cancel query selector and only get Cancel buttons.

        • bleistift2@sopuli.xyz
          link
          fedilink
          English
          arrow-up
          2
          ·
          6 days ago

          This only works if the cancel button looks like a specialized button. If it doesn’t (for instance, if it looks like the usual “×” symbol), you’re in for a lot of style overrides.

          Oh, and guess what: Your suggestion is exactly what bootstrap does. .btn.primary is for the default action, .btn.secondary I usually use for aborting and going back, .btn.danger shows that bad things will happen. The only difference is that if doesn’t force its default styles on all plain <button> elements that might be present on the page.

          • dohpaz42@lemmy.world
            link
            fedilink
            English
            arrow-up
            3
            ·
            6 days ago

            You’re absolutely right. My idea is to imply that developers should give their semantics actual thought and intention.

            For your example, the X button can be called .dismiss, since typically that’s what clicking the X does; whereas cancelling something may have more meaning (e.g. cancelling a subscription, which triggers other actions aside of dismissing a modal window).

    • Eager Eagle@lemmy.world
      link
      fedilink
      English
      arrow-up
      2
      ·
      6 days ago

      Besides what the other comment mentioned, that’s also not how most front-end frameworks work. You often need specific classes with your elements to use the default styles.