Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Sample AngularHint application? #36

Open
ealtenho opened this issue Aug 1, 2014 · 5 comments
Open

Sample AngularHint application? #36

ealtenho opened this issue Aug 1, 2014 · 5 comments
Assignees

Comments

@ealtenho
Copy link
Contributor

ealtenho commented Aug 1, 2014

@btford Besides the contrived examples in the e2e tests, we had discussed having a more realistic sample application using AngularHint to demonstrate its functionality. Would this sample application be developed in this repository? Do we have any ideas of what sort of application it should be like?

@ealtenho ealtenho self-assigned this Aug 5, 2014
ealtenho pushed a commit that referenced this issue Aug 6, 2014
@ealtenho
Copy link
Contributor Author

ealtenho commented Aug 6, 2014

@caguillen214, I've created an example app that is meant to look somewhat correct although underneath it violates many best practices in order to showcase how silent errors affect applications. Please feel free to add to the errors in the application or change it as necessary. I know that many of the errors in HintDirectives correspond to custom directives, but I wasn't sure if it was overkill to create a custom directive for this simple application. What do you think? I'm also considering making a second application with identical look and feel and goal, but that does not violate the best practices.

@ealtenho
Copy link
Contributor Author

ealtenho commented Aug 6, 2014

In 17ed6ae I've added a correctExample that has the same behavior as the example but does not violate best practices. It should not produce any error messages. However, some problems in individual modules cause incorrect hints. I've filed issues on those modules: https://github.com/angular/angular-hint-dom/issues/14, https://github.com/angular/angular-hint-modules/issues/14, https://github.com/angular/angular-hint-controllers/issues/10

@caguillen214
Copy link

This is great! And I think thats a good idea. I do think that making custom directives would be beneficial though because theres a lot of features related to that in HintDirectives

@ealtenho
Copy link
Contributor Author

ealtenho commented Aug 7, 2014

Okay, I've basically spent today fixing the issues I referenced above that I discovered in testing the SampleApplication. I will work on adding a custom directive to test and show AngularHintDirectives behavior.

ealtenho pushed a commit that referenced this issue Aug 7, 2014
Add good and bad uses of custom directives in example and correctExample
to show more ngHint options for directives.

#36
@ealtenho
Copy link
Contributor Author

ealtenho commented Aug 7, 2014

In 149b9b5 I updated the incorrect example app and the correctExample app to use a custom directive. Honestly, I'm not sure I'm familiar enough with directives to make a really convincing broken example. Feel free to suggest changes. I've tried to maximize the number of hints that I am generating.

@ealtenho ealtenho mentioned this issue Aug 7, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants