-
-
Notifications
You must be signed in to change notification settings - Fork 7k
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
fix (v-edit-dialog): restores save/open/close/cancel events #3227
Conversation
Since we have to wait for v1.1 to implement this, maybe it's something we could look at to improve even more. I know you have expressed dislikes in the way we are currently handling this. |
I'm not a big fan of having another component that is de facto the v-dialog with some predefined props, some magic behind and limited possibility of customizing. |
It's a menu though isn't it? And yeah I'm cool with releasing this in 1.1 but also deprecating the component at the same time, with examples of how to implement it manually of course. |
Codecov Report
@@ Coverage Diff @@
## dev #3227 +/- ##
==========================================
- Coverage 86.87% 86.73% -0.15%
==========================================
Files 158 158
Lines 4199 4206 +7
Branches 1332 1332
==========================================
Hits 3648 3648
- Misses 442 449 +7
Partials 109 109
Continue to review full report at Codecov.
|
19d7ecc
to
073f412
Compare
Description
adds back VEditDialog events that were removed in 1.0
Motivation and Context
fixes #3197
How Has This Been Tested?
docs
Markup:
Types of changes
Checklist:
master
for bug fixes,dev
for new features and breaking changes).Docs
https://github.com/vuetifyjs/vuetifyjs.com/pull/269