Skip to content

Offer contextual action to delete Widget #1756

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.

Sign up for GitHub

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

Jump to bottom
Closed
matthew-carroll opened this issue Feb 9, 2018 · 5 comments
Closed

Offer contextual action to delete Widget #1756

matthew-carroll opened this issue Feb 9, 2018 · 5 comments
Assignees
@scheglov
Milestone
M22

Comments

@matthew-carroll
Copy link

Feature Request:
Currently there are contextual actions to create new Widgets as children, or wrap an existing Widget with a new parent.

It would be equally helpful to have a contextual action to delete a Widget from a given build() configuration. It can be very tedious to try to extract a Widget in the middle of a deep tree.
@devoncarew
Copy link
Member

You mean something like delete a widget, but keep it's children?

/cc @scheglov, @mit-mit

@matthew-carroll
Copy link
Author

Yes, and also keep its parent. For example:

Widget1
--Widget2
----Widget3 <--- DELETE THIS ONE
------Widget4

@scheglov
Copy link
Contributor

scheglov commented Feb 9, 2018

Same as #1283 (comment), right?

@matthew-carroll
Copy link
Author

Yeah I think #1283 is the same thing.

@scheglov scheglov self-assigned this Feb 9, 2018
@scheglov
Copy link
Contributor

scheglov commented Feb 9, 2018

https://dart-review.googlesource.com/c/sdk/+/40240

@devoncarew devoncarew added this to the M22 milestone Feb 9, 2018
@scheglov scheglov closed this as completed Feb 9, 2018
dart-bot pushed a commit to dart-lang/sdk that referenced this issue Feb 9, 2018
@scheglov
New Quick Asists for Flutter: 'Replace with child' and 'Replace with …
59716fe 
…children'.

[email protected], [email protected]

Bug: flutter/flutter-intellij#1283
Change-Id: Ic50de9739cdab5cd82649dcbd8370c5e728d2ee3
     flutter/flutter-intellij#1756
Reviewed-on: https://dart-review.googlesource.com/40240
Reviewed-by: Brian Wilkerson <[email protected]>
Reviewed-by: Devon Carew <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@scheglov scheglov

Labels
None yet
Projects
None yet
Milestone
M22
Development

No branches or pull requests
3 participants
@scheglov @devoncarew @matthew-carroll