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

showDropDown broken, should use ShareDialogView #19214

Closed
PVince81 opened this issue Sep 21, 2015 · 9 comments
Closed

showDropDown broken, should use ShareDialogView #19214

PVince81 opened this issue Sep 21, 2015 · 9 comments

Comments

@PVince81
Copy link
Contributor

@blizzz can you take care of this ?

It should be a matter of opening the dropdown and embedding the new ShareDialogView into it.
It seems it doesn't work any more.

CC @oparoz

@PVince81
Copy link
Contributor Author

The gallery app needs this

@karlitschek
Copy link
Contributor

Yes. This is critical

@blizzz
Copy link
Contributor

blizzz commented Sep 23, 2015

After investigating, some apps (at least Gallery :D) were using the old share dialog and manipulating it. Best and safest way for 8.2 seems to pack the "old" share dialog code into one file, let it both ship and load by the apps who need it. For 9.0 we look into a way to satisfy the needs of gallery, calendar and such by providing possibilities to interact with the new share dialogue code in a sane way (real API :D).

@PVince81
Copy link
Contributor Author

I'm looking into providing a portable namespaced "share.js" that can be copied into gallery, calendar and contacts and usable directly.

@PVince81
Copy link
Contributor Author

I tried putting these additions into the gallery app and it worked.
But the checkboxes are broken (the new styles), so let's wait for @Henni to fix them in owncloud/calendar#940 first.

@PVince81 PVince81 assigned PVince81 and unassigned blizzz Sep 24, 2015
@PVince81
Copy link
Contributor Author

PRs are available now for calendar, contacts and gallery: #19214 (comment)

Please review them!

@VicDeo
Copy link
Member

VicDeo commented Sep 29, 2015

Related to owncloud-archive/documents#550

@PVince81
Copy link
Contributor Author

PVince81 commented Oct 1, 2015

All known apps have been fixed => closing

@PVince81 PVince81 closed this as completed Oct 1, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Aug 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants