adds new edit flow for the audio block#15520
Closed
draganescu wants to merge 7 commits intomasterfrom
Closed
Conversation
draganescu
requested review from
Soean,
ajitbohra,
gziolo,
jorgefilipecosta,
noisysocks,
notnownikki,
talldan and
youknowriad
as code owners
This was referenced May 8, 2019
talldan
reviewed
May 9, 2019
talldan
added
[Block] Audio
Contributor
|
From a design perspective, this is working well on my end:
|
gziolo
added
the
Good First Review
draganescu
force-pushed
the
update/new-edit-flow-audio
branch
from
3bf1a04 to
bdc4f16
Compare
Contributor
Author
|
Updated with the review actions from @talldan and rebased. |
talldan
reviewed
May 24, 2019
Contributor
|
Works great for me! Thanks @draganescu!! |
draganescu
force-pushed
the
update/new-edit-flow-audio
branch
from
de4d56f to
051f005
Compare
Contributor
Author
|
rebased and refactored according to @talldan 's suggestion. |
Contributor
Author
|
#11952 changed direction so closing this as it became irrelevant. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Closes #14795
This is a follow up to the image block edit flow update which ports the new flow to all the blocks which have media with an edit state.
How has this been tested?
For now I've only tested locally.
Types of change
New feature (non-breaking change which adds functionality)
Screenshots