material2-app icon indicating copy to clipboard operation
material2-app copied to clipboard

Updated to @angular/material 2.0.0-beta.6 and @angular/* 4.1.3.

Open SamuelMarks opened this issue 7 years ago • 3 comments

Your repository was linked from Google's as the featured example. Unfortunately you have lapsed behind their codebase.

I've brought you back up to speed, specifically I've:

  1. Generated a new scaffold from @angular/cli 1.0.3;
  2. Implemented @angular/router routing with lazy loading;
  3. Followed the official @angular/material install guide;
  4. Used step 3 suggestion to create one module for all @angular/material module imports;
  5. Migrated the relevant code from your example

Enjoy

SamuelMarks avatar May 30 '17 15:05 SamuelMarks

Hy Samuel,

i've seen your updates / PR but I'm not so happy with them. What I like is that you invested the effort to update it t the latest material / cli level.

however this is a beginners material 2 sample so why add ed routing ? (this makes it more complicated to understand.) also: change the src dir to m2-app ? (same reason) you also included the now unused app.component default files
and missed some material-modules in the import-module. (sidenav / toolbar)

(anyway it seems this sample is not maintained anymore (if looking to the issues)

Greetings from germany - michael

m67hoff avatar May 31 '17 14:05 m67hoff

@m67hoff - Okay, I've:

  • removed routing;
  • removed extra module;
  • added missing CSS & scripts;
  • imported MdToolbarModule

… and called out—@jelbourn—the maintainer for review.

EDIT: Just updated to Node.JS 8.0.0 with npm 5.0.0; so added package-lock.json. Also some minor fixes that the previous commit left/broke.

SamuelMarks avatar May 31 '17 14:05 SamuelMarks

I hadn't really intended this app to live forever; IMO it's been superseded by https://github.com/angular/material.angular.io at this point. If you want to fork the repo, though, to serve as your own example you're totally welcome to it.

I'll update the README to reflect the current status, and probably stop linking here from the material2 repo

jelbourn avatar May 31 '17 16:05 jelbourn