Skip to content

Conversation

@youknowriad
Copy link
Contributor

This tiny PR fixes adding a caption to an image block with an empty caption. The edited caption was not saved because we're destroying the editor before the onFocusOut (that triggers onChange) is being called, thus, the value was not saved.

You can experience the same bug with the quote citation.

@youknowriad youknowriad added the [Type] Bug An existing feature does not function as intended label Apr 24, 2017
@youknowriad youknowriad self-assigned this Apr 24, 2017
@youknowriad youknowriad requested a review from jasmussen April 24, 2017 09:22
@jasmussen
Copy link
Contributor

Seems to work great!

This is also possibly a thing for a separate PR as this is also an issue with the image caption. But there's no placeholder text in the empty citation field:

screen shot 2017-04-24 at 11 29 46

It should say "Write citation..." in $light-gray-500 ;)

@youknowriad
Copy link
Contributor Author

But there's no placeholder text in the empty citation field

Right! waiting for #475 to fix this.

@youknowriad youknowriad merged commit c3e7e54 into master Apr 24, 2017
@youknowriad youknowriad deleted the fix/update-image-caption branch April 24, 2017 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants