Basic.css icon indicating copy to clipboard operation
Basic.css copied to clipboard

Text fields in articles aren't visible

Open getaaron opened this issue 2 years ago • 1 comments

The background color of an article is the same as the background color of a text field, and the text field doesn't have a border, so the text field becomes invisible when putting a text field inside an article:

image

One possible approach is to add a border, for example in this diff:

-input, select, textarea {font-size: 1em; color:var(--c4); background: var(--c2); border: 0; padding: 0.6em}
+input, select, textarea {font-size: 1em; color:var(--c4); background: var(--c2); border: 1px solid var(--c4); padding: 0.6em}

which adds a border:

image

However, I didn't check any other use cases so it's likely this is undesirable in some way.

If it's helpful, here's some reproduction HTML

<section>
  <article>
    <form action="/weather" accept-charset="UTF-8" method="post">
      <h2>New weather search</h2>
      <div>
        <label for="zip_code">Zip code</label>
        <input type="text" name="zip" id="zip">
        <div>
          <input type="submit" name="commit" value="Search Weather" data-disable-with="Search Weather">
        </div>
      </div>
    </form>
  </article>
</section>

getaaron avatar Jan 16 '23 02:01 getaaron

I may not be using this properly, but as far as I can see, the text has no margin which makes it harder to read. Adding perhaps a few pixels of margin could improve it unless I'm missing something Screenshot 2024-02-02 at 23 45 04

FinancialMistake avatar Feb 02 '24 23:02 FinancialMistake