Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Comments

feat: add ability to set code editor settings#2552

Merged
cwhitten merged 23 commits intomicrosoft:masterfrom
a-b-r-o-w-n:feat/code-editor-options
Apr 8, 2020
Merged

feat: add ability to set code editor settings#2552
cwhitten merged 23 commits intomicrosoft:masterfrom
a-b-r-o-w-n:feat/code-editor-options

Conversation

@a-b-r-o-w-n
Copy link
Contributor

@a-b-r-o-w-n a-b-r-o-w-n commented Apr 7, 2020

Description

  • Allows users to set code editor options in the settings page.
  • Adds ability to adjust size of dialog nav and property editor
  • Persists these settings to local storage
  • Fixes an issue on settings page where the active page was not highlighted when refreshing

Task Item

closes #2385
closes #987

Screenshots

image

image

Screen Recording 2020-04-07 at 11 40 AM

@hibrenda
Copy link
Contributor

hibrenda commented Apr 8, 2020

#987

@a-b-r-o-w-n a-b-r-o-w-n deleted the feat/code-editor-options branch April 17, 2020 17:16
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* set up code editor options in client store

* add ability to change editor options at runtime

* add codeEditorSettings to shell data

* wire up code editor settings to lg/lu editors

* add userSettings namespace

* add ability to resize property editor

* allow changing settings in all up views

* add minimap to code editor settings

* remove settings cog

* move code editor settings to preferences page

* add ability to change dialog nav width

* remove unused import

* fix up settings routes

* remove dead code

* update lockfile

* fix onboarding e2e spec

* remove cy.wait in favor of asserting on the url

* add additional assertion to wait for homepage to load

Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
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.

3 participants