Skip to content

Use new modal w/ detail dialog for compiler list in Walkthrough#7887

Merged
Colengms merged 5 commits intomainfrom
coleng/compiler_list_modal_detail
Jul 29, 2021
Merged

Use new modal w/ detail dialog for compiler list in Walkthrough#7887
Colengms merged 5 commits intomainfrom
coleng/compiler_list_modal_detail

Conversation

@Colengms
Copy link
Contributor

@Colengms Colengms commented Jul 28, 2021

VS Code added a new modal dialog which displays details less prominently than the main message. This change switches to using that UI for the detected compiler list that can be displayed via the Getting Started walkthrough.

It was (on Windows):
image

It's now (on Windows):
image

@jureid
Copy link
Contributor

jureid commented Jul 29, 2021

Let's shorten the message now that it's a heading. Can we try this:

"We found the following C++ compilers on your system. Choose a compiler in your project's IntelliSense Configuration."

I don't think we need the parentheses in compiler(s). If the list only contains 1, I think it still reads well.

@Colengms Colengms merged commit 590dd67 into main Jul 29, 2021
@Colengms Colengms deleted the coleng/compiler_list_modal_detail branch July 29, 2021 22:07
@github-actions github-actions bot locked and limited conversation to collaborators Sep 13, 2021
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