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

Image alt text in markdown widget defaults to null #956

Closed
erquhart opened this issue Dec 21, 2017 · 2 comments · Fixed by #1778
Closed

Image alt text in markdown widget defaults to null #956

erquhart opened this issue Dec 21, 2017 · 2 comments · Fixed by #1778

Comments

@erquhart
Copy link
Contributor

- Do you want to request a feature or report a bug?

bug

- What is the current behavior?

When an image is inserted in the markdown editor (rich text mode), the alt text is persisted as null if left blank.

- If the current behavior is a bug, please provide the steps to reproduce.

  • create a new entry that uses the markdown widget
  • insert an image before making any other changes in the widget
  • switch to markdown view
  • note that alt value is "null" as a string

- What is the expected behavior?

  • alt value should be an empty string

- Please mention your CMS, node.js, and operating system version.

CMS 1.0.3

@papandreou
Copy link
Contributor

This still happens in 1.8.1 after #1126 was merged.

@erquhart
Copy link
Contributor Author

Confirmed, thanks for pointing that out.

leonardodino added a commit to leonardodino/netlify-cms that referenced this issue Sep 28, 2018
leonardodino added a commit to leonardodino/netlify-cms that referenced this issue Sep 29, 2018
leonardodino added a commit to leonardodino/netlify-cms that referenced this issue Sep 29, 2018
leonardodino added a commit to leonardodino/netlify-cms that referenced this issue Sep 29, 2018
leonardodino added a commit to leonardodino/netlify-cms that referenced this issue Sep 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants