Skip to content

Start the article text level with the buttons - #48

Merged
bonelifer merged 1 commit into
mainfrom
fix/article-body-padding
Oct 13, 2026
Merged

bonelifer merged 1 commit into
mainfrom
fix/article-body-padding

Conversation

@bonelifer

Copy link
Copy Markdown
Contributor

On an article page the first line of the text sat above the buttons on the right: the button row has a top margin of about 0.8em, and the text started at the top of the panel.

The text area now has that room as top padding (0.8em), and the button row's top margin is removed. The result:

  • The text starts level with the buttons.
  • The buttons stay exactly where they were, at the same position as before on phpBB 3.3.19 and on 4.0.0-a3-dev. I measured both.

Only styles/prosilver/theme/style.css changes.

Tested

  • Screenshots of an article page on phpBB 3.3.19 and on 4.0.0-a3-dev, before and after, with the buttons' position measured. The first line sits on the buttons' row, and the buttons are unmoved.
  • Unit tests: 217 tests, 498 assertions, all pass.
  • Not run on MySQL or PostgreSQL. The change is one CSS rule, with no PHP or database code.

Investigated and written by Claude on behalf of William Jacoby (bonelifer).

The first line of an article sat above the buttons on the right. The text area now has
the room the buttons had as a top margin, so the text starts level with them and the
buttons stay where they were.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@bonelifer
bonelifer merged commit 56a1f70 into main Oct 13, 2026
58 checks passed
@bonelifer
bonelifer deleted the fix/article-body-padding branch October 13, 2026 00:18
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.

1 participant