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

Update to Angular 4, Fix ngSemantic is not a Module #178

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Update to Angular 4, Fix ngSemantic is not a Module #178

wants to merge 9 commits into from

Conversation

mp3sii
Copy link

@mp3sii mp3sii commented Apr 30, 2017

Good day vladotesanovic.
I made a few changes in order to fix and update ngSemantic with new angular. This is related with the issue of ngSemantic is not a module. #149

This is the list of the changes:

  • Fixed ngSemantic is not a module by using @borisfeldmann [solution] (NgSemantic is not a module #149 (comment))

  • Changed and updated dependencies. Mainly:

    • Used Angular cli@"^1.0.1", instead of SystemJs
    • Updated to Angular 4
    • Updated other dependencies
  • Fixed errors due to upgrade angular4 and cli, these included errors in Tab and Accordion

    • Zone.js in demo was updated to a newer version
    • Removed Directive in Accordion, instead using the Accordion Component to initiate Jquery and input options
    • Removed Directive call in Tab. (The demo works fine, but I may have caused other issues that I cannot foresee at this moment)

Some TODOs:

  • Remove SystemJs references

This is my first pull request ever, and I would understand if you decide to implement this changes on your own. Hopefully this would help you.
Thank you for this project

- Fixed ngSemantic is not a module by using [@borisfeldmann](https://github.com/borisfeldmann) [solution](#149 (comment))
- Updated dependencies to more up to date angular
- Fixed errors due to new angular version like dependencies on Tab and Accordion
    - Zone.js in demo
    - Removed Directive in Accordion, instead using the Accordion Component to initiate Jquery and input options
@mp3sii mp3sii changed the title ## version: 1.2 Update to Angular 4, Fix ngSemantic is not a Module Apr 30, 2017
@gridcellcoder
Copy link

@mp3sii really need this fix but it seems this project is stalled since December 2016.

@mbrookson
Copy link

Would also be keen to see this resolved

@landru29
Copy link

What is the blocking point to fix this major issue ?

@vlados
Copy link

vlados commented Nov 7, 2017

@vladotesanovic any plans to make it work with Angular 4?

@landru29
Copy link

landru29 commented Nov 7, 2017

Now I'm using "ng2-semantic-ui" package which works perfectly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants