a11y: resolve outstanding issues in Notifications subpage #2342
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* first commit of working prototype for mongoDB storage * change mechanism for creating zip file for syncing to runtime now builds all the files loaded into project (from arbitrary source) rahter than zipping a folder on disk * clean up console output * plugin loader looks for plugins, finds them * dynamically load in the storage provider via plugin loader use custom storage if one is specified * remove some debug traces * clean up some minor code formatting things * disable linter rule to allow for requiring in plugin modules * slight refactor of plugin loader to encapsulate functionality add sample web route plugin first tiny stab at publish -- TIME TO BRANCH * functioning auth implementattion using passportjs * sample github auth * first round of changes for multi-user support Related to #1918 and #1917 * add in the bot project id into the route * fix new bot * fix save as * rationalize all the URLS used in context of a project * plumb user identity down through storage * disable plugins * add readme * plugin readme * plugin readme * plugin readme * plugin readme * import publisher controller from other branch * update to readme * import prototype of publishing page * updates * delete environment and connector * add error checking for missing project by id * Better error handling for attempts to write to a missing or moved project Should fix last remaining item in #1918 * add publish method with localpublish plugin * keep bot dir clean * basic publish of localpublish plugin * remove hostBot * add gitignore file * change unzip template into copy template folder * Fix build script * Fix default projectMap not exists * change luis operation to match local plugin * polish default setting * use projectId * clean up * remove lib dir * use projectId * use limit 'currentBotProjects' cache * remove unused * Update copy * return error response when publish failed * update name initialization process * fix some test,hide publish tab and change deploy script * fix UT integrity check * fix bf-cli version error * update the version * remove @microsoft/bf-cli-command@1.0.0 * try to fix todo e2e test * fix lint * fix lint * remove some unused test and skip some failed unit tests * feat: Prevent old content from overwriting newer changes (#2156) * updates to write operations to prevent overwrite not 100% working on LU files yet * debounce lu * remove comment * remove some console logs * address feedback from chris * remove commented code * change hint word deploy to publish * fix lint * fix some test syntax * update the ui for notification page * fix luis publish path not found * fix some e2e tests * fix onboarding when using projectId * fix notification page bug * Back out simple overwrite protection, will replace with less error prone mechanism * add plugins volumn in docker * fix docker up * fix unit test * fix the filesetting path * change docker to map multi ports. in order to support multi runtime * add projectId when create and delete dialog * fix botstatus * fix botstatus * remove console log * fix settings files path * update the moke data Co-authored-by: Wenyi Luo <wenyluo@microsoft.com> Co-authored-by: Dong Lei <donglei@microsoft.com> Co-authored-by: zhixzhan <zhixzhan@microsoft.com> Co-authored-by: liweitian <liweitian93@outlook.com> Co-authored-by: leilzh <leilzh@microsoft.com> Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
* add cosmos db container template * replace deprecated interface Co-authored-by: Qi Kang <qika@microsoft.com> Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
* a11y: remove extra Link stuff from settings Nav * remove stray console.logs Co-authored-by: Dong Lei <donglei@microsoft.com>
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
* auto update data.json when schema updated * fix lint * add a version number and use generic migration to do re-init
Co-authored-by: Andy Brown <asbrown002@gmail.com>
* replace IconButton with Icon Replaces the IconButton in WidgetLabel.tsx with a plain Icon bearing an aria-label. * fix unit tests * Update WidgetLabel.tsx Add a keyboard tab stop and adjust spacing - still not sure the SR experience is 100% great, but this at least makes it work. Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
* fix issues 2193 * fix issue 2094 add heading * add header in about page Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
… sync (#2173) * move `useWindowDimensions` to separated file * move `useLgTemplate` under hooks folder * use `lgFiles` in shell store to compute lg text * move useLgTemplate to 'hooks' folder Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
* fix🐞(uischema): fix choices string According to sdk.schema, the 'choices' field of 'Microsoft.ChoiceInput' could be either object list or string list. Make Visual Editor support both of those formats. #2319 * fix🐞(uischema):dynamic choices According to sdk definition, 'choices' could be an expression string. Co-authored-by: Andy Brown <asbrown002@gmail.com>
Co-authored-by: Andy Brown <asbrown002@gmail.com>
In 'Microsoft.SetProperties', the 'value' field of an assignment could be either object or string. Stringify the value if it's an object. #2309 Co-authored-by: Andy Brown <asbrown002@gmail.com>
I have added language parameter for generating the luconfig.json file to allow generation in another language. For example is-is. Also I have added property control to make it 'es-us' by default Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com> Co-authored-by: Andy Brown <asbrown002@gmail.com>
This addresses a few minor problems that were causing a11y issues.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This is a set of small changes that address the ARIA needs on the Notifications page.
Task Item
Closes #2124
Screenshots