-
Notifications
You must be signed in to change notification settings - Fork 72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implemented moving pictures within their frame #39
base: master
Are you sure you want to change the base?
Conversation
Nice, thanks for this feature! However, I don't like the activation via Ctrl: it is an easter egg that is very hard to discover. What about merging |
Having this feature is great. Why not merge as is, so the user at least has the option to move the pic in the frame, then in a separat new issue merge @jdavidberger is it possible to zoom in/out as well? |
Because it's best to avoid changes in user interfaces. On non-beta software, users don't want to start using a feature, then get lost because it is not accessible the same way anymore (for instance with Plus, the change I ask is relatively easy :)
I agree this would be very nice! (But this can go in another pull request.) |
I'll see if I can get to it this week. It isn't a difficult modification;
the only reason I didn't do it like this originally was in an effort to not
effect existing behaviors.
It would be nice, long term, to see something like redraw on mouse move.
The current render loop is somewhat incompatible with that; I think you'd
have to do away with the 'rendering' pop up and probably also try to have
it only draw the one cell that needed it to scale well with big collages.
…On Thu, Dec 29, 2016 at 1:31 AM, Adrien Vergé ***@***.***> wrote:
Why not merge as is
Because it's best to avoid changes in user interfaces. On non-beta
software, users don't want to start using a feature, then get lost because
it is not accessible the same way anymore (for instance with Ctrl).
Plus, the change I ask is relatively easy :)
is it possible to zoom in/out as well?
I agree this would be very nice! (But this can go in another pull request.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#39 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AH0LCySzAXtsMEPOgDalk_PqU4XbApEQks5rM2_bgaJpZM4LVUK_>
.
|
Moved the file chooser dlg of "Add images" into own class. Added an image preview widget to dialog. Code is based on https://github.com/endlessm/eos-photos/blob/bc787c9f43a99576fcdfca15505dd8438424aeef/src/widgets/preview_file_chooser_dialog.py
fix #39 Add image preview to file chooser
Sorry, this was automatically closed because of "fix #39" in @frankMilde pull request #41. Reopening. |
@jdavidberger I'd love to have this feature! Please, if you changed the event condition already, could you please share it? |
is this feature already in master branch ? |
See #10