akveo / akveo/blur-admin

make directives global

Đang mở
#151 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
JavaScript
Star
11.3k
Fork
3k
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

Hi,

this is feedback I just want to give you. Though I am not an experienced front-end developer I could easily get into the sample code of the application. First I thought, wow there's not much documentation, but if you make the tutorial on "how to create a new page" and look into the samples it's quite easy to figure out how it works.

But here's one thing I'd really love to see. The samples use directives as a part of the specific module. But it would be a lot nicer if they were in a directives package on the same level as the pages package. Then you can just depend on it. For example I did something like this:
`angular.module('BlurAdmin.pages.jobs, ['BlurAdmin.directives.selectpicker']) {`

This way i have a dedicated module namespace for directives and don't make it look like my module has dependencies on other modules, with functionality i don't need.

Sincerely,
Marco

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.