Skip to content

Conversation

@Markus-SWAG
Copy link
Contributor

No description provided.

@keymanapp-test-bot
Copy link

User Test Results

Test specification and instructions

ERROR: user tests have not yet been defined

@keymanapp-test-bot keymanapp-test-bot bot added this to the A19S22 milestone Feb 2, 2026
@github-project-automation github-project-automation bot moved this to Todo in Keyman Feb 2, 2026
@Markus-SWAG Markus-SWAG enabled auto-merge February 2, 2026 10:04
Comment on lines 82 to 89
Copy link
Member

@mcdurdin mcdurdin Feb 2, 2026

Choose a reason for hiding this comment

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

I think the example is actually incorrect; the intent of the description was correct, and the example as it stands was unhelpful (why would someone want to type the vowel twice and then delete the second one when ^ was pressed?)

Suggested change
side of a rule. If a repeating vowel is found, then this rule adds a circumflex
on top of both vowels when <key>^</key> is pressed.
```
store(vowel) "aeiou"
store(vowelcirc) "âêîôû"
any(vowel) index(vowel,1) + "^" > index(vowelcirc,1) index(vowelcirc,1)
```

Copy link
Contributor Author

Choose a reason for hiding this comment

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

corrected as suggested

@Markus-SWAG Markus-SWAG force-pushed the fix/docs/fix-index-example branch from d1d63c8 to 13cf296 Compare February 2, 2026 22:32
@Markus-SWAG Markus-SWAG force-pushed the fix/docs/fix-index-example branch from 13cf296 to 37f692d Compare February 2, 2026 22:51
@Markus-SWAG Markus-SWAG merged commit 515ebc0 into master Feb 2, 2026
3 of 5 checks passed
@Markus-SWAG Markus-SWAG deleted the fix/docs/fix-index-example branch February 2, 2026 23:52
@github-project-automation github-project-automation bot moved this from Todo to Done in Keyman Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants