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

Feature/add edit button #1566

Closed
wants to merge 2 commits into from
Closed

Feature/add edit button #1566

wants to merge 2 commits into from

Conversation

marimeireles
Copy link

WIP: #1524

So @camilasan can check my progress :)

… emit the sign like the code was supposed to.

Signed-off-by: Mariana Meireles <mariana@psychonautgirl.space>
Signed-off-by: Mariana Meireles <mariana@psychonautgirl.space>
@marimeireles marimeireles force-pushed the feature/add-edit-button branch from 5bf9005 to f6feaf5 Compare October 27, 2019 15:17
@camilasan camilasan self-requested a review November 4, 2019 19:15
@camilasan camilasan added this to the 2.7 🌟 UI improvements milestone Nov 4, 2019
@jancborchardt
Copy link
Member

@marimeireles is this based on what @camilasan did at #1442 or does it replace it? Or is both needed? :)

@marimeireles
Copy link
Author

As far as I'm concerned this is a different thing. Gonna try to work on this now, sorry, I was on vacation and lost track of things.

@marimeireles
Copy link
Author

But wait a second.. @camilasan is this still relevant for the new NextCloud desktop version?

@jancborchardt
Copy link
Member

@marimeireles yeah it should be, or why not? :) Are you referring to Virtual drive, or the new tray menu? Neither of those make a right-click → Edit functionality obsolete, as far as I can see. :)

@er-vin
Copy link
Member

er-vin commented Sep 16, 2020

Doesn't apply anymore on master, a new approach would need to be taken. Please feel free to open a new PR related to that and sorry for that lost effort...

@er-vin er-vin closed this Sep 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants