Skip to content

Update the NV Newspage screenshots for its new layout - #34

Merged
bonelifer merged 1 commit into
mainfrom
newspage-ui-refresh-screenshots
Oct 19, 2026
Merged

bonelifer merged 1 commit into
mainfrom
newspage-ui-refresh-screenshots

Conversation

@bonelifer

@bonelifer bonelifer commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Updates the NV Newspage screenshots in phpbb-test-board/screenshots.py for its refreshed layout (phpbbmodders/phpbb-ext-newspage#3).

  • A fourth example news topic opens with a picture. The script draws it with PHP's GD into the board's images/ folder, using the screenshot server's address in the BBCode, and deletes it afterwards.
  • The main shots turn on the new "Show the first image above the news" setting.
  • The single news item is taken as the admin, so "Reply" shows.
  • A new shot, newspage-news-filtered, shows the news filtered by forum. The shortened shot turns the lead image off.
  • The cleanup list now deletes files as well as folders; it used shutil.rmtree only, which skipped files silently.
  • phpbb-test-board/README.md describes the new set.

Claude ran it on the local phpBB 3.3.19 test board for phpbb-ext-newspage's ui-refresh branch: 8 saved, 0 failed, and the drawn picture was removed afterwards.


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

🤖 Generated with Claude Code

Adds a fourth news topic opening with a picture the script draws with
GD (and deletes afterwards), turns on the lead image for the main shots,
takes the single news item as the admin, and adds a shot filtered by
forum. The cleanup list now removes files as well as folders.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bonelifer
bonelifer merged commit 25b52d9 into main Oct 19, 2026
1 check passed
@bonelifer
bonelifer deleted the newspage-ui-refresh-screenshots branch October 19, 2026 19:26
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