littlefox/neokom
Watch
1
2
Fork
You've already forked neokom
 1
Code Issues 8 Pull requests Projects Releases 12 Packages Activity Actions

Say how a picture is changed to fit the server (#359) #391

Merged
littlefox merged 1 commit from issue-359-say-changes into main 2026-10-28 14:23:06 +00:00
Conversation 0 Commits 1 Files changed 11 +290 -45
littlefox commented 2026-10-28 13:58:52 +00:00
Owner
Copy link

Closes #359

Per the user's comment on #359 (D-072): instead of chasing the black square, say how a picture is changed to fit a lyskomd text. The composer, the chat's reply field and the profile editor list, under the picture: the original's size against the limit, pixels before and after, format and JPEG quality, transparent parts turned white, and a lost animation. A picture that goes as it is says nothing.

Verified: unit tests for GIF frame counting and APNG/WebP animation; e2e gif.spec (new test with a half-transparent 2000 px PNG, plus the too-big GIF's list), compose, profile and thread-chat specs, 51/51 in desktop, Android and iPhone profiles; lint three ways.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XJW2ocwLunyaLiUwCBHEP8

Closes #359 Per the user's comment on #359 (D-072): instead of chasing the black square, say how a picture is changed to fit a lyskomd text. The composer, the chat's reply field and the profile editor list, under the picture: the original's size against the limit, pixels before and after, format and JPEG quality, transparent parts turned white, and a lost animation. A picture that goes as it is says nothing. Verified: unit tests for GIF frame counting and APNG/WebP animation; e2e `gif.spec` (new test with a half-transparent 2000 px PNG, plus the too-big GIF's list), compose, profile and thread-chat specs, 51/51 in desktop, Android and iPhone profiles; lint three ways. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01XJW2ocwLunyaLiUwCBHEP8
Say how a picture is changed to fit the server
All checks were successful
perf / perf (pull_request) Has been skipped
Details
ci / plan (push) Successful in 1m52s
Details
ci / check (push) Successful in 9m22s
Details
ci / desktop (push) Successful in 16m16s
Details
ci / e2e (android) (push) Successful in 20m15s
Details
ci / e2e (iphone) (push) Successful in 20m20s
Details
ci / e2e (desktop) (push) Successful in 21m51s
Details
bf5511ca67 
The user would rather know how a picture is altered than chase #359's
black square further (D-072). When the composer, the chat's reply field
or the profile editor redraws a picture, it now lists what changed: the
original's size against the server's limit, the pixels before and after,
the format and JPEG quality, transparent parts turned white, and a lost
animation (GIF frames counted, APNG and animated WebP recognised). A
picture that goes as it is says nothing.

Refs #359

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XJW2ocwLunyaLiUwCBHEP8
littlefox referenced this pull request from a commit 2026-10-28 15:15:38 +00:00
Release 0.8.0
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
area/ci
Forgejo Actions, runner, tests

  
area/desktop
Desktop build

  
area/packaging
Arch, Debian, Fedora, NixOS packages

  
area/protocol
kom-proto, wire format, charset

  
area/reading
Unread tracking, reading order, review

  
area/ui
Dioxus app, layout, interaction

  
area/web
Web build and relay

  
area/writing
Composing and sending texts

  
kind/bug
Something is broken

  
kind/feature
New functionality

  
kind/infra
Tooling, tests, CI, dev environment

  
kind/request
requests for features or changes from users

  
needs/user
Waiting for a decision or action from littlefox

  
parked
Kept for reference; no work planned until the user says so

  
perf
Run the perf workflow and its budgets on this pull request (#287)

  
security
Security-relevant

  
upstream
A proposal for Protocol A or lyskomd, for the LysKOM development group

No labels
area/ci
area/desktop
area/packaging
area/protocol
area/reading
area/ui
area/web
area/writing
kind/bug
kind/feature
kind/infra
kind/request
needs/user
parked
perf
security
upstream
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
littlefox/neokom!391
No description provided.