Skip to content

Various visual improvements#122

Merged
jholdstock merged 4 commits into
decred:masterfrom
jholdstock:max1000
Jun 13, 2026
Merged

Various visual improvements#122
jholdstock merged 4 commits into
decred:masterfrom
jholdstock:max1000

Conversation

@jholdstock

Copy link
Copy Markdown
Member

Looks like a large diff but its a lot of whitespace changes.

See individual commits for details.

Previously the page was filling 80% of the available window with no
maximum, looking ridiculous on ultrawide screens. This commit adds a
sensible maximum (1000px).
- Prevent longer articles taking up too much homepage space by only
  showing a preview (first 500 words). Add a "read more" link to the
  full article if its longer than 500 words.
- Increase spacing between articles so they are more obviously
  separated.

This makes the template for the homepage and for the /news page
sufficiently different for them to be separated rather than configured
with params.
Nowhere else are slashes surrounded by spaces, so make these cases
consistent with the rest of the site text.
The emergence of the sloptimism problem can clearly be seen by charting the number of reports submitted to the Decred bug bounty program since its inception.

<img src="/img/report-chart.svg" style="margin: auto; max-width: 700px; width 90%;" alt="chart"/>
<img src="/img/report-chart.svg" style="margin: auto; max-width: 700px; width: 90%;" alt="chart"/>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was probably a fun one to find. Easy to miss.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was a few minutes of head-scratching thats for sure. Very annoying.

<div>
<a class="see-more" href="{{ .Site.Params.news.URL | absURL }}">See all news and updates...</a>
</div>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not that it matters, but this seems like an extra blank line unbalanced with the opening tag and anchor that doesn't have one.

@jholdstock jholdstock merged commit 1e39307 into decred:master Jun 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants