Skip to content

Update js and css libraries consumed by theme layout file - #16

Closed
atiq-cs wants to merge 2 commits into
statiqdev:mainfrom
atiq-cs:layout-packages
Closed

Update js and css libraries consumed by theme layout file#16
atiq-cs wants to merge 2 commits into
statiqdev:mainfrom
atiq-cs:layout-packages

Conversation

@atiq-cs

@atiq-csatiq-cs commented Nov 11, 2021

Copy link
Copy Markdown
Contributor

update all js and css reference packages that are outdated

Following are not touched,

  • lunr doesn't have an update
  • extras.scss

Test plan

PRs are stacked on top of other. This is the final output.

Check output as well,

index page

image

post page

image

@atiq-csatiq-cs changed the title update all packages in _layout.cshtml except lunrUpdate js packages in theme layout fileNov 11, 2021
 update all packages in `_layout.cshtml` except lunr which doesn't have an update
@atiq-csatiq-cs changed the title Update js packages in theme layout fileUpdate js and css libraries in theme layout fileNov 12, 2021
@atiq-csatiq-cs changed the title Update js and css libraries in theme layout fileUpdate js and css libraries consumed by theme layout fileNov 12, 2021
@atiq-cs
atiq-csforce-pushed the layout-packages branch 4 times, most recently from 8034a2c to 3ec841cCompareNovember 12, 2021 08:05
@atiq-cs

atiq-cs commented Nov 23, 2021

Copy link
Copy Markdown
ContributorAuthor

Note: Trivial update, please have a look at individual commits inside this PR for further technical details. Cheers!

@daveaglick

Copy link
Copy Markdown
Member

Same question here - is there still something here that we should merge on top of #23?

@atiq-cs

Copy link
Copy Markdown
ContributorAuthor

For layout packages update part 1, ref: 2f5a5ac

note for myself,

@atiq-cs

Copy link
Copy Markdown
ContributorAuthor

For layout packages update part 2: cleanblog js and scss to v6.0.7, ref: 3ec841c
has been taken care of by 1eeabd4, thanks @rdeago

@atiq-csatiq-cs closed this Dec 6, 2022
@daveaglick

Copy link
Copy Markdown
Member

🎉

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@atiq-cs@daveaglick