robrix icon indicating copy to clipboard operation
robrix copied to clipboard

Add image viewer widget

Open alanpoon opened this issue 4 months ago • 9 comments

Fixes #327 Screenshot 2025-08-06 at 5 46 55 PM

in replacement of https://github.com/project-robius/robrix/pull/443

Waiting for this PR to be merged: https://github.com/makepad/makepad/pull/788.

alanpoon avatar Aug 06 '25 09:08 alanpoon

Intention of adding an image magnifier and the ability to pan image.

The image's image_scale, image_pan will always be set to default in draw_walk. Hence unable to dynamically scale and pan image. https://github.com/makepad/makepad/issues/785

alanpoon avatar Aug 11 '25 04:08 alanpoon

Intention of adding an image magnifier and the ability to pan image.

The image's image_scale, image_pan will always be set to default in draw_walk. Hence unable to dynamically scale and pan image. makepad/makepad#785

@alanpoon I just checked with Rik, and this line should not be present in the Image widget. It's some errant code left over from dealing with animations in a strange way.

You can submit a PR to makepad that removes that line, and then continue with this issue here.

kevinaboos avatar Oct 17 '25 09:10 kevinaboos

Sure, the makepad PR is here: https://github.com/makepad/makepad/pull/788/

alanpoon avatar Oct 18 '25 02:10 alanpoon

Screenshot 2025-10-19 at 10 58 51 AM Screenshot 2025-10-19 at 10 58 57 AM Screenshot 2025-10-19 at 10 59 17 AM

alanpoon avatar Oct 19 '25 03:10 alanpoon

Requires https://github.com/makepad/makepad/pull/806 to be merged to use the rotated_image widget

alanpoon avatar Oct 30 '25 07:10 alanpoon

Requires makepad/makepad#806 to be merged to use the rotated_image widget

I've asked Rik to review & merge this.

kevinaboos avatar Oct 30 '25 21:10 kevinaboos

Requires makepad/makepad#806 to be merged to use the rotated_image widget

I've asked Rik to review & merge this.

@alanpoon this PR has been merged, and I've also updated the main Robrix branch to depend on the latest makepad.

kevinaboos avatar Oct 30 '25 23:10 kevinaboos

@alanpoon i finally fully fixed all Modal issues in Makepad, or at least I think I did. PR #632 should resolve all of your modal problems; please let me know if it works or if there are still some remaining problems.

kevinaboos avatar Nov 13 '25 02:11 kevinaboos

@alanpoon i finally fully fixed all Modal issues in Makepad, or at least I think I did. PR #632 should resolve all of your modal problems; please let me know if it works or if there are still some remaining problems.

I have merged main into my branch. There is no more event propagation to the beneath layer. I don't think there is anymore outstanding issues. Please kindly review.

alanpoon avatar Nov 13 '25 04:11 alanpoon