Skip to content

Fix: Applying blur does not apply pending rotations. - #6932

Merged
RitikaPahwa4444 merged 1 commit into
commons-app:mainfrom
rovertrack:fixblur
Aug 2, 2026
Merged

Fix: Applying blur does not apply pending rotations.#6932
RitikaPahwa4444 merged 1 commit into
commons-app:mainfrom
rovertrack:fixblur

Conversation

@rovertrack

@rovertrack rovertrack commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Description (required)

context: #6920 (review)

What changes did you make and why?

  • applypendingrotation required file to be passed but inside the method it just needed the imageUri to be converted to file object so moved that logic to function itself as the imageUri is global it will be available for all to use.
  • added a method call to applypendingrotation after the image blur happens, this saves us from calculating and remapping the entire post blurred image to compare at runtime, as after blur rotating results in the same file.

Screenshots (for UI changes only)
Before:

az_recorder_20260802_014250.mp4

After:

az_recorder_20260802_013950.mp4

@github-actions

github-actions Bot commented Aug 1, 2026

Copy link
Copy Markdown

✅ Generated APK variants!

@RitikaPahwa4444
RitikaPahwa4444 merged commit 7378359 into commons-app:main Aug 2, 2026
2 of 3 checks passed
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