Skip to content

Conversation

@dfsm
Copy link
Contributor

@dfsm dfsm commented Nov 15, 2025

  • New option added "Forum Attachment".
  • Menu Tree updated to handle new option.
    Now [img] will prompt:
    a) (If clipboard not empty) Paste URL
    b) From Library (from Camera has been removed)
    If From Library is selected, user can select their image. Once selected, another pair of options is presented:
    a) Image Host (if imgurUploadsEnabled)
    b) Forum Attachment
  • If image is too large (dimensions or filesize) (likely!) then option to Resize & Continue is presented. User can cancel out of resize to meet reqs.
  • Preview card UI element appears during resizing and also to display attachment thumbnail, dimensions and filesize in newreply view
  • Preview card with Keep or Delete segmented options present when editing a post with an attachment
  • Fixed existing issue where attachments would not render in posts. Had to download these with authentication and then display them in the posts after the html renders

I have pushed this through Claude 13 times for refactoring passes. This ought to be squeaky clean!!!

dfsm added 13 commits November 10, 2025 20:48
- New option added "Forum Attachment".
- Menu Tree updated to handle new option.
Now [img] will prompt:
a) (If clipboard not empty) Paste URL
b) From Library (from Camera has been removed)
If From Library is selected, user can select their image. Once selected, another pair of options is presented:
a) Image Host (if imgurUploadsEnabled)
b) Forum Attachment
- If image is too large (dimensions or filesize) (likely!) then option to Resize & Continue is presented. User can cancel out of resize to meet reqs.
- Preview card UI element appears during resizing and also to display attachment thumbnail, dimensions and filesize in newreply view
- Preview card with Keep or Delete segmented options present when editing a post with an attachment
- Fixed existing issue where attachments would not render in posts. Had to download these with authentication and then display them in the posts after the html renders
@dfsm dfsm requested a review from nolanw November 15, 2025 05:46
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