Replies: 4 comments 7 replies
-
@trusktr -- This is a good list. Thanks for putting it together and tackling much of the work. I think it's important to recognize that while this type of cleanup work will benefit maintainers, the end result of the items listed above do not provide a compelling reason for existing users to upgrade from v4. This isn't meant as a negative comment. I think Docsify needs this work to be done and that it will be in a much better place when this work is completed. My intention is only to point out that if we are going to introduce changes that will require a major version bump, we should also consider changes that directly benefit users so that they are incentivized to upgrade. One suggestion is update the UI visuals, HTML, and CSS for v5. If proposals for v5 include converting all of our CSS from Stylus to vanilla CSS and converting our HTML from string/templates to UI components then this would be an opportune time to do it. By incorporating lessons learned from docsify-themeable and popular component libraries we can offer an updated UI with flexible theming options. Since this change would be highly visible, I believe this change alone would motivate users to upgrade. A few quick thoughts regarding other items on the list above:
Ping: @sy-records, @Koooooo-7 |
Beta Was this translation helpful? Give feedback.
-
I added a new one to the list above:
|
Beta Was this translation helpful? Give feedback.
-
Added a new bullet to the list above:
|
Beta Was this translation helpful? Give feedback.
-
I fantasize about removing the build. For example, we could just have a The less we have to maintain for a working product, the more we can advance features, and we can let others help us with code or sponsorship. Regarding the Vue stuff, I would really like to make that a standalone plugin rather than core feature. |
Beta Was this translation helpful? Give feedback.
-
We have some baggage,
.styl
syntax for our CSS. CSS has lots of new language features, including CSS Nesting which is out in all browsers (Firefox with a flagnow in Firefox without a flag), container queries, etc.I propose:
v5
(Any chained pull requests are in the same order as in this list)
.call
, but rather be private methods. refactor: move some functions and module-level state into classes as private methods and properties to start to encapsulate Docsify #2136var
->const
, orfor(;;)
->for(of)
, so it will be easier to review them sepatelyFuture release if not v5
Non breaking changes:
Potentially Breaking changes:
/index.html
anddocs/index.html
. Do we need duplicate code? Delete one.initGlobalAPI()
along with the automaticnew Docsify
and instead require users to pass options intonew Docsify(options)
)using postcss for now only until CSS Nesting lands in FirefoxFirefox supports nesting now). These changes should be backwards compatible for docsify-themeable. Someone with a build system might consume these files, so major version bump.Beta Was this translation helpful? Give feedback.
All reactions