Skip to content

Conversation

@diegoreymendez
Copy link
Contributor

Description

Fixes #709

It seems we were missing a call to edited().

Testing

  1. Launch the app
  2. Type anything using the Japanese keyboard.
  3. Switch to HTML mode and edit some more.
  4. Switch back to visual mode and make sure all Japanese characters are properly rendered throughout the session.

Copy link
Contributor

@bummytime bummytime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works like a champ @diegoreymendez. Nice job! Just a nit with some extraneous spaces. :shipit:

}
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a nit...could we remove these spaces?

@diegoreymendez
Copy link
Contributor Author

@bummytime - Done. I'll wait for Travis to confirm and then merge.

@diegoreymendez diegoreymendez merged commit 510d841 into develop Nov 15, 2017
@diegoreymendez diegoreymendez deleted the issue/709-japanese-text-issues branch November 15, 2017 15:54
@diegoreymendez diegoreymendez changed the title Fixes an issue with non english characters. Fixes a rendering issue with non english characters. Nov 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants