Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: select for demo Sketch files #43

Merged
merged 3 commits into from
Sep 25, 2018

Conversation

gioboa
Copy link
Contributor

@gioboa gioboa commented Sep 6, 2018

Implementation for #21

Changes:

  • now with a select you can choose from different demo Sketch files

@manekinekko manekinekko added this to To do in Beta.0 via automation Sep 7, 2018
@gioboa
Copy link
Contributor Author

gioboa commented Sep 17, 2018

Hi @manekinekko,
is there something wrong in my implementation?
Do you need some fix? 😉

})
export class SketchSelectDemoFilesComponent implements OnInit {
public selectedDemoFile = '';
public demoFiles = [
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe we should move this list to the sketch.service.ts. the SketchSelectDemoFilesComponent should remain as dump as possible. WDYT?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@manekinekko Yep. It's more cleaner. 👍

@manekinekko manekinekko merged commit 8781d5f into xlayers:develop Sep 25, 2018
Beta.0 automation moved this from To do to Done Sep 25, 2018
@ghost ghost removed the in progress label Sep 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Beta.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants