Skip to content

[Release]: v4.11.4 - #1190

Merged
anikethsaha merged 81 commits into
masterfrom
develop
Jun 18, 2020
Merged

[Release]: v4.11.4#1190
anikethsaha merged 81 commits into
masterfrom
develop

Conversation

@anikethsaha

@anikethsahaanikethsaha commented May 22, 2020

Copy link
Copy Markdown
Member

Do Not Merge


Changelog
  • ...
Blocking

image

image

Any rollup expert can tackle this ?

snyk-botand others added 30 commits March 31, 2020 21:31
…4c39a1be1464a76e
[Snyk] Upgrade docsify from 4.10.2 to 4.11.2
[fix#1115] Image resize in % doesn't work.
* fix
* Update search.js
* Update search.js
* Update search.js
* Update search.js
* Update search.js
Update embeded files documentation
Add example with front matter
Add various comments I used to understand code
Updated front matter to have installed flag
Added function to parse front matter
Added intercept to remove front matter for embed
Also, add a link explaining why that can be a security issue.
Fix typo on `externalLinkRel`
…ility
Improve accessibility of viewport meta by allowing page zoom (related to #945)
Updated embed to strip YAML front matter fixes#1129
@codesandbox-ci

codesandbox-ciBot commented Jun 8, 2020

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 714ef29:

SandboxSource
ecstatic-lake-wse4xConfiguration

trusktrand others added 2 commits June 8, 2020 00:45
…ned search bar using docsify-darklight-theme" (#1207)
* Revert "update: updated docs site dark and light mode with switch and redesigned search bar using docsify-darklight-theme (#1182)"
This reverts commit 415f295.
* docs: removed the old dark mode as well
@trusktr

Copy link
Copy Markdown
Member

it has been merged and it will be shipped with this release.

Great Thanks !

@boopathikumar018 The change was reverted, as @jhildenbiddle brought up some good points, along with a breaking change in IE 11. (we need to improve PRs to have testing for all browsers). See there for some ideas: hoping you can implement a subset of the PR with an option for users to specify light and dark themes, then your theme changes can come in following PR.

trusktrand others added 3 commits June 12, 2020 21:37
* develop:
chore: update auto format config
Co-authored-by: James George <jamesgeorge998001@gmail.com>
* Fix cover overlapping sidebar
* Fix transition of cover page
* Revmove transition property from .cover
* Remove z-index attribute from _coverpage.styl
Co-authored-by: Joe Pea <joe@trusktr.io>
@anikethsaha

Copy link
Copy Markdown
MemberAuthor

All blockers are fixed.

Ready to release once CI is green.

@anikethsaha

Copy link
Copy Markdown
MemberAuthor

@trusktr@jhildenbiddle@Koooooo-7@sy-records we have a new blocker

@trusktr

Copy link
Copy Markdown
Member

I could've sworn I added a test to ensure the bundle is loaded properly. Need to improve that test. See docsify.test.js

@trusktr

Copy link
Copy Markdown
Member

This is the file: https://www.github.com/docsifyjs/docsify/tree/develop/test%2Funit%2Fdocsify.test.js

npm test does a prod build then runs that test. If that test is passing, it means the bundle loads fine, but maybe it fails once we try it do something on the site?

We can use window.addEventListener('error', ...) to observe uncaught errors at the top level (f.e. in the test file).

@trusktr

Copy link
Copy Markdown
Member

Didn't it work in the preview? And tests passed? Did you perhaps need to run npm install? Or does this happen even after install? And if so, then how did the PR pass testing?

@anikethsaha

Copy link
Copy Markdown
MemberAuthor

Thanks for the heads up @trusktr , I actually had 2-3 docsify code opened in my machine, ran npm install in one, ran the build command in other 😄

@trusktr

Copy link
Copy Markdown
Member

Hehe. Happens to the best of us. I can't count how many times I've done something like that. ;)

@anikethsaha
anikethsaha merged commit 714ef29 into masterJun 18, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

15 participants

@anikethsaha@trusktr@boopathikumar018@snyk-bot@jamesgeorge007@CHU295@Koooooo-7@sy-records@vagra@HerbertHe@sytone@emersonreisdev@kumaravel95@maubuz@mohammedsahl