Skip to content

Revert "Remove loweredDigits6dots from Spanish Character Definitions"#1623

Closed
jmdaweb wants to merge 2 commits into
liblouis:masterfrom
nvda-es:revert-1574-fix-spanish-chardefs
Closed

Revert "Remove loweredDigits6dots from Spanish Character Definitions"#1623
jmdaweb wants to merge 2 commits into
liblouis:masterfrom
nvda-es:revert-1574-fix-spanish-chardefs

Conversation

@jmdaweb

@jmdaweb jmdaweb commented Aug 24, 2024

Copy link
Copy Markdown
Contributor

This reverts #1574 because the change causes the number sign being removed from digits on spanish grade 1 table.

@jmdaweb

jmdaweb commented Aug 24, 2024

Copy link
Copy Markdown
Contributor Author

CC: @ivnc

@bertfrees bertfrees added the tables Something that needs to be fixed in table files label Aug 24, 2024
Comment thread tests/braille-specs/es-g2.yaml
@jmdaweb

jmdaweb commented Aug 24, 2024

Copy link
Copy Markdown
Contributor Author

Because I just pressed the revert button on #1574. Do you want to keep the test?

@bertfrees

Copy link
Copy Markdown
Member

Yes. Unless you say it is wrong, or could be improved.

@egli

egli commented Aug 26, 2024

Copy link
Copy Markdown
Member

I tried to fix this PR but I do not have permission. First off I made sure the original copyright statement in the YAML test is kept. Secondly I marked the test as xfail for backward as the problem outlined in the test is no longer fixed with the revert.

@jmdaweb can you give me permission to push to you branch?

@egli egli added this to the 3.31 milestone Aug 26, 2024
@jmdaweb

jmdaweb commented Aug 26, 2024

Copy link
Copy Markdown
Contributor Author

Of course. Now you should be able to write.

@bertfrees

Copy link
Copy Markdown
Member

We should merge #1613 first.

@bertfrees bertfrees added the needs fixup Branch needs cleaning up before it's merged. Don't press any buttons! label Aug 26, 2024
@egli

egli commented Aug 27, 2024

Copy link
Copy Markdown
Member

Fixed in #1624

@egli egli closed this Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs fixup Branch needs cleaning up before it's merged. Don't press any buttons! tables Something that needs to be fixed in table files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants