MaterialDesignInXamlToolkit icon indicating copy to clipboard operation
MaterialDesignInXamlToolkit copied to clipboard

Update Getting Started

Open Keboo opened this issue 6 years ago • 5 comments

Right now the Getting Started section in the README is a bit outdated.

People who are just getting started with this library are not likely to have all of the dependencies needed (latest version of VS, .NET Core, etc.) to compile the source code and demo application.

I am looking for people newer to this project to either comment or submit a PR with suggestions on how to improve this.

Some ideas to consider:

  • Getting started section so that it is as simple as possible for someone new to the library.
  • ~Consider including a link to my examples repo~
  • Consider moving up the link to the releases page with the pre-compiled demo app from the See It In Action section
  • Remove the Source Code section and instead simply link over to the wiki page

Keboo avatar Sep 20 '19 20:09 Keboo

I think it is important to list the executable demo first before mentioning the compilable demo code. "here, you can try this. It runs out of the box. And if you want to compile it yourself: here is the code, but you need this and that".

The wiki ends with a link to the demo code. A link to the executable could be included right before the code link, i think: https://github.com/MaterialDesignInXAML/MaterialDesignInXamlToolkit/wiki/Getting-Started

Taschenbuch avatar Sep 22 '19 08:09 Taschenbuch

The Configuring your Windows section has a duplicate section within the same page.

The duplicate sections each use a different BackgroundBrush in their examples but are otherwise identical. One of these should be removed.

BJohnsonDev avatar Sep 26 '19 21:09 BJohnsonDev

The Configuring your Windows section has a duplicate section within the same page.

The duplicate sections each use a different BackgroundBrush in their examples but are otherwise identical. One of these should be removed.

Oh, I was just going to create a new issue about this duplication. I am glad I found this issue.

FWIW, the duplication was added in this commit and a diff of those two sections (using line numbers relative to that first line in each section) is

7c7
<         Background="{DynamicResource MaterialDesignBackground}"
---
>         Background="{DynamicResource MaterialDesignPaper}"
16c16
<         Background="{DynamicResource MaterialDesignBackground}"
---
>         Background="{DynamicResource MaterialDesignPaper}"

where the the left/first file in the diff is the first occurance of the "Configuring your Window(s)" section (so the right/second file in the diff is the second occurance of the section).

TysonMN avatar Nov 13 '19 14:11 TysonMN

Is this still something that needs done?

Christian-Oleson avatar Sep 10 '21 02:09 Christian-Oleson

@Christian-Oleson yes, i have been busy with other items and haven't had a change to go back to it (though some of the other docs have been updated). It is specifically about the main README page and the items under the Getting Started heading.

Keboo avatar Sep 10 '21 06:09 Keboo