tbolt / boltcss Goto Github PK

These elements were not touched in the initial cut and should be updated or removed.

Looks like the assets from the demo contribute a fair bit of weight to the npm package. Consider not including them in the package with a .npmignore file?

Not a huge deal, but the 5+MB weight for a simple prototyping library caught my eye

Issue

When using a Markdown processor (like e.g. Hugo), the following code block:

```bash
npm create hyas@latest
```

gets rendered into html like so:

<pre><code class="language-bash">npm create hyas@latest
</code></pre>

and displays as:

Looks like the extra border was not intended.

Solution

Possible solution is to update pre > code to something like:

pre > code {
  padding: 8px 6px;
  border: 0;
  display: block;
  overflow-x: auto;
}

Resources

• Code Blocks

Not sure if this is intentional (or already fixed, I saw something in #4 related to it), but reporting in case it’s unintended…

Thanks for your library, definitely something I’ll want to use!

I was evaluating how good it looks on the phone and thought it looked great except the space on the top reduced the amount of content on the front page and looked a bit out of place.

Luckily these aren't styles in bolt itself but on the demo page.

Do you think reducing the margin to half or so on mobile with a media query would be an improvement?

.demo-body {
    max-width: 800px;
    margin: 40px auto;

I looked at it some more and I might have changed my mind a bit, but I want to suggest it just in case. Maybe bringing it down to 30px on mobile would make the first impression nicer even though it wouldn't look like much to most comparing them.

Even though bolt.css is a small stylesheet there should be a minified version

Ensure pointer and keyboard navigation work well with default form / element highlighting