Skip to content
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

Copy/cut an image triggers the fullscreen view #829

Closed
enzus opened this issue Aug 18, 2015 · 2 comments · Fixed by plone/plone.app.content#50
Closed

Copy/cut an image triggers the fullscreen view #829

enzus opened this issue Aug 18, 2015 · 2 comments · Fixed by plone/plone.app.content#50

Comments

@enzus
Copy link

enzus commented Aug 18, 2015

Just tested on http://dev.brianledwell.com:8081/Plone/image.jpg/view

If I click on Cut or Copy I'm redirected to the fullscreen view of the image (the one without any view template applied) although the action has been performed correctly.

@pbauer pbauer added this to the Plone 5 milestone Aug 19, 2015
@pbauer
Copy link
Member

pbauer commented Aug 19, 2015

@enzus thanks for reporting. Keep the tickets coming 😃

@pbauer
Copy link
Member

pbauer commented Aug 20, 2015

The same is true for rename

pbauer added a commit to plone/plone.app.content that referenced this issue Aug 20, 2015
vangheem added a commit to plone/buildout.coredev that referenced this issue Aug 20, 2015
Branch: refs/heads/master
Date: 2015-08-20T11:14:29+02:00
Author: Philip Bauer (pbauer) <[email protected]>
Commit: plone/plone.app.content@347a2a1

fix plone/Products.CMFPlone#829

Files changed:
M plone/app/content/browser/actions.py
M plone/app/content/tests/test_actions.py
Repository: plone.app.content
Branch: refs/heads/master
Date: 2015-08-20T09:29:44-05:00
Author: Nathan Van Gheem (vangheem) <[email protected]>
Commit: plone/plone.app.content@a38f30b

Merge pull request #50 from plone/respect_view_method

Respect view-methods in rename, copy and cut

Files changed:
M plone/app/content/browser/actions.py
M plone/app/content/tests/test_actions.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants