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

"Two-word briefs You *" lesson fixes #525

Merged

Conversation

paulfioravanti
Copy link
Contributor

This PR proposes a range of dictionary fixes (details in each commit message) that came up for me working through Typey Type's Two-word briefs You * lesson.

- Mark `KOU` for "you could" as a mis-stroke
- Add `KOU` for "cow" to `dict.json`
- Change Gutenberg dictionary outline preference for "cow" to `KOU`
 - Mark `TKU` for "you did" as a mis-stroke
 - Correct `TKU` entry in `dict.json` to be "did you"
fix the entries in `dict.json` and mark their original outlines as
mis-strokes.
- Mark `TKPWU` for "you go" as a mis-stroke
- Fix `TKPWU` entry in `dict.json` to be "Gu"
- Mark `SRU` for "you have" as a mis-stroke
- Fix `SRU` entry in `dict.json` to have value "have you"
- Mark `URL` for "you recall" as a mis-stroke
- Fix `URL` entry in `dict.json` to have value "URL"
Therefore,

- Mark `URBGD` for "you recollected" as a mis-stroke
- Fix `URBGD` entry in `dict.json` to have value "understanding"
…/HR`.

Therefore,

- Add `HR*U` for "you will" entry to `dict.json`
- Mark `HRU` for "you will" as a mis-stroke
- Fix `HRU` entry in `dict.json` to have value "will you"
- Move `U/HR` for "you will" entry from `dict.json` to condensed strokes
@didoesdigital
Copy link
Owner

Oof, this lesson looks like a mess. This may need some surgery to rebuild the lesson properly in light of the many changes in Plover dictionary. I'll need to come back to this.

Thanks for the PR, @paulfioravanti

@didoesdigital didoesdigital merged commit 53c4640 into didoesdigital:master Dec 17, 2020
@paulfioravanti paulfioravanti deleted the two-word-briefs-you-fixes branch December 17, 2020 10:08
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

2 participants