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

Fix #5267 #5508

Closed
wants to merge 2 commits into from
Closed

Fix #5267 #5508

wants to merge 2 commits into from

Conversation

vrdhn
Copy link

@vrdhn vrdhn commented Nov 18, 2015

This patch reads out commonly auto-converted bullet types in Word.
It converts these symbol from PUA to Unicode, and adds these to symbols.dic

@jcsteh
Copy link
Contributor

jcsteh commented Nov 18, 2015

Looks like you have to reference it in the comment, rather than the summary. So, this is for #5267.

@dkager
Copy link
Collaborator

dkager commented Jan 18, 2017

Alternative solution: instead of replacing the PUA symbols with other Unicode symbols, we could replace them with the ASCII characters you type to get them. E.g. =>, <>, etc.
Advantage is that they are more likely to appear in the user's braille table.

@dkager
Copy link
Collaborator

dkager commented Jan 21, 2017

This looks like a good resource for mapping from the Wingdings font Word uses for some bullets to standard Unicode. Remove the 0xF0 prefix from the PUA codes and look them up in the table. Unfortunately, this doesn't quite match what is done in the PR. I will discuss it also with sighted users.

@dkager
Copy link
Collaborator

dkager commented Jan 22, 2017

In Word you can also Save As --> Plain text --> UTF-8. This way Word transforms PUA codes into Unicode, although the result is not unambiguous. E.g. U+F0B7 and U+F0A8 are both transformed into U+2022 (bullet). Still, in my opinion it is a good idea to do what Word itself does.

The only problem is that you can no longer identify each individual list level, because some will share the same bullet symbol. But this is already a problem if you make a list with more than 3 levels, because then the bullets start repeating.

@dkager
Copy link
Collaborator

dkager commented Aug 6, 2017

Closing this because #6778 supercedes it. This provided a valuable basis for the other PR, but I don't think we will be merging this one as-is. However, please reopen if I am overlooking something.

@dkager dkager closed this Aug 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants