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

module.directive('FirstLetterCapitalized', ...) does not work and does not log an error. #11109

Closed
jamestalmage opened this issue Feb 20, 2015 · 1 comment
Closed

module.directive('FirstLetterCapitalized', ...) does not work and does not log an error. #11109

jamestalmage opened this issue Feb 20, 2015 · 1 comment
Labels
component: $compile frequency: low severity: confusing type: bug
Milestone
Backlog - won't fix

Comments

@jamestalmage
Copy link
Contributor 

//app.js
angular.module('myModule', []).directive('MyDirective', function() {
  console.log('Hello World!');
  /* ....  make a directive .... */
});
<!--- index.html -->
<my-directive></my-directive>

This produces absolutely zero output on the console. Changing the directive name to start with a lower case letter fixes the problem. IMO, it should either log an error that I capitalized the first letter, or just work.
@pkozlowski-opensource
Copy link
Member

Yeh, kind of corner case, but you are right...

@pkozlowski-opensource pkozlowski-opensource added this to the Backlog milestone Feb 20, 2015
@wesleycho wesleycho mentioned this issue Mar 9, 2015
Closed
petebacondarwin pushed a commit that referenced this issue Mar 17, 2015
@wesleycho @petebacondarwin
fix($compile): throw error on invalid directive name
170ff9a 
Directive names must start with a lower case letter.
Previously the compiler would quietly fail.
This change adds an assertion that fails if this is not the case.

Closes #11281
Closes #11109
petebacondarwin pushed a commit that referenced this issue Mar 17, 2015
@wesleycho @petebacondarwin
fix($compile): throw error on invalid directive name
634e467 
Directive names must start with a lower case letter.
Previously the compiler would quietly fail.
This change adds an assertion that fails if this is not the case.

Closes #11281
Closes #11109
netman92 pushed a commit to netman92/angular.js that referenced this issue Aug 8, 2015
@wesleycho @netman92
fix($compile): throw error on invalid directive name
8be4f95 
Directive names must start with a lower case letter.
Previously the compiler would quietly fail.
This change adds an assertion that fails if this is not the case.

Closes angular#11281
Closes angular#11109
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
component: $compile frequency: low severity: confusing type: bug
Projects
None yet
Milestone
Backlog - won't fix
Development

No branches or pull requests
2 participants
@pkozlowski-opensource @jamestalmage