Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

improve command provider #1981

Merged
merged 2 commits into from
Jun 18, 2018
Merged

improve command provider #1981

merged 2 commits into from
Jun 18, 2018

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Jun 16, 2018

Fixes element-hq/element-web#5069

Fixes second part of element-hq/element-web#5467

when typing a slash command, it would be nice if the autocomplete box stuck around after you typed the command name, since the autocomplete box shows the syntax of the command

improve Regexp so it leaves autocomplete up whilst typing arguments
improve completion so it doesn't discard your arguments on-hover
add a way to list all commands by using `/<TAB>`

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@@ -2,6 +2,7 @@
Copyright 2016 Aviral Dasgupta
Copyright 2017 Vector Creations Ltd
Copyright 2017 New Vector Ltd
Copyright 2018 Michael Telatynski <7t3chguy@gmail.com>
Copy link
Member

Choose a reason for hiding this comment

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

Not sure if this should be New Vector

Copy link
Member Author

Choose a reason for hiding this comment

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

This wasn't a p1 issue and thus not chargeable hours so <Me/>

@dbkr
Copy link
Member

dbkr commented Jun 18, 2018

Otherwise lgtm

@t3chguy t3chguy assigned dbkr and unassigned t3chguy Jun 18, 2018
@dbkr dbkr merged commit 1ab902d into develop Jun 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants