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

Migrate project to angular-cli #1738

Merged
merged 10 commits into from
Sep 4, 2019
Merged

Migrate project to angular-cli #1738

merged 10 commits into from
Sep 4, 2019

Conversation

marjan-georgiev
Copy link
Member

What kind of change does this PR introduce? (check one with "x")

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

What is the new behavior?
Migrates the project to angular-cli
Builds using ng-packagr
Ivy compatible
Removes the compiled files from the git repository
Reformats the whole codebase

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

@luisb90
Copy link

luisb90 commented Sep 4, 2019

found a disparity issue while running through the demos.
http://swimlane.github.io/ngx-datatable/#server-scrolling
in the above, when you scroll down, it shows a blue loading bar across the bottom of the table, as more records are loaded. Locally, while the records do load, the loading bar does not show.

Screen Shot 2019-09-04 at 11 22 18 AM

screenshot shows blue bar im referring to.

@luisb90 luisb90 closed this Sep 4, 2019
@luisb90 luisb90 reopened this Sep 4, 2019
@marjan-georgiev
Copy link
Member Author

Fixed - needed to update the style in that demo.

@marjan-georgiev marjan-georgiev merged commit aab2dbe into master Sep 4, 2019
@bmesin
Copy link

bmesin commented Sep 10, 2019

Are you sure it works now?

image

@hugoj-goncalves
Copy link

Are you sure it works now?

image

even on wannabebrowser (I.E. 9) seems fine to me...

@marjan-georgiev
Copy link
Member Author

Are you sure it works now?

image

What are you testing on?

This is what I see:

image

@bmesin
Copy link

bmesin commented Sep 10, 2019

I usually use Chrome and it's ok there, but this was on Safari.

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.

7 participants