Skip to content

Add script for the Telegram web interface#9

Merged
michaelDCurran merged 2 commits intomasterfrom
telegram
Jan 17, 2017
Merged

Add script for the Telegram web interface#9
michaelDCurran merged 2 commits intomasterfrom
telegram

Conversation

@michaelDCurran
Copy link
Copy Markdown
Contributor

So far just automatically reports incoming messages.

Copy link
Copy Markdown
Owner

@jcsteh jcsteh left a comment

Choose a reason for hiding this comment

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

Please rename to TelegramA11yFixes.user.js. I know the suffix seems superfluous, but I'm concerned users might get confused if this was distributed in isolation. Thanks!

readme.md Outdated

This script improves the accessibility of the [Telegram instant messaging](https://web.telegram.org/) web interface.

it so far does the following:
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Capital i at start of sentence.

readme.md Outdated
This script improves the accessibility of the [Telegram instant messaging](https://web.telegram.org/) web interface.

it so far does the following:
- marks the chat history as a live region so new messages are announced automatically
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

  1. Need a blank line before this because of GitHub Markdown weirdness.
  2. Capital at start, full stop at end.

@jcsteh
Copy link
Copy Markdown
Owner

jcsteh commented Jan 17, 2017

Oh, and copyright year and version should be 2017, not 2016. :)

@michaelDCurran michaelDCurran merged commit 6b95fa2 into master Jan 17, 2017
michaelDCurran added a commit that referenced this pull request Jan 20, 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.

2 participants