Skip to content

fix: apply transform to screencopy buffer - #125

Closed
Gigas002 wants to merge 1 commit into
waycrate:masterfrom
Gigas002:fix/freeze-rotation
Closed

fix: apply transform to screencopy buffer#125
Gigas002 wants to merge 1 commit into
waycrate:masterfrom
Gigas002:fix/freeze-rotation

Conversation

@Gigas002

@Gigas002 Gigas002 commented Aug 3, 2026

Copy link
Copy Markdown
Member

Mirrors a similar fix in wayshot: waycrate/wayshot#411
Applied only when running with --freeze arg

@Gigas002 Gigas002 self-assigned this Aug 3, 2026
@Decodetalkers

Decodetalkers commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

it is not needed anymore I think. just update the libwayshot to rc version will fix it

That pr was for it . I notice this bug, and because that issue, so I made that fix and pr. Now the saved image will always in normal direction, just update libwayshot to the 0.9.0-rc1

@Gigas002

Gigas002 commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

Ah, I see. Tested and it's indeed works good with -rc ver. Will update toml files after we release 0.9 then

@Gigas002 Gigas002 closed this Aug 3, 2026
@Gigas002

Gigas002 commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

@Decodetalkers
Interesting enough, when running wayshot -g $"(waysip -d --freeze)" using either this branch or libwayshot's -rc ver, I get correct rendering of my transformed output.
Using wayshot -g as is produces rotated output though...

@Decodetalkers

Decodetalkers commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

@Decodetalkers Interesting enough, when running wayshot -g $"(waysip -d --freeze)" using either this branch or libwayshot's -rc ver, I get correct rendering of my transformed output. Using wayshot -g as is produces rotated output though...

Because wayshot -g is using the wl_buffer and do transform once a time.

But waysip always places buffer in normal directions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants