Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

fix(angular): move styles import to angular.json #532

Merged
merged 2 commits into from
Oct 21, 2021
Merged

fix(angular): move styles import to angular.json #532

merged 2 commits into from
Oct 21, 2021

Conversation

dhayab
Copy link
Member

@dhayab dhayab commented Oct 19, 2021

With the Angular template, importing styles located in node_modules using @import do not work in Codesandbox environments. To work around this, I moved the imports to angular.json where it is handled correctly.

I also added basic styling to remove margin/padding on html and set a sans-serif font.

This should close #531.

@Haroenv
Copy link
Contributor

Haroenv commented Oct 19, 2021

the snapshots still need to be updated @dhayab, but this looks promising!

.circleci/config.yml Outdated Show resolved Hide resolved
@dhayab dhayab force-pushed the fix/531 branch 2 times, most recently from 1af8ba9 to ee5d3a3 Compare October 19, 2021 13:12
@dhayab dhayab marked this pull request as draft October 19, 2021 14:33
@dhayab dhayab marked this pull request as ready for review October 20, 2021 11:37
@dhayab dhayab requested review from a team, tkrugg and francoischalifour and removed request for a team October 20, 2021 11:37
@dhayab dhayab merged commit d2f73b8 into master Oct 21, 2021
@dhayab dhayab deleted the fix/531 branch October 21, 2021 07:43
aymeric-giraudet pushed a commit to algolia/instantsearch that referenced this pull request Dec 14, 2022
…antsearch-app#532)

* fix(angular): move styles import to angular.json
* test(angular): update template snapshots
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

angular template doesn't import css correctly
3 participants