nlbdev / nordic-epub3-dtbook-migrator

Tools for converting between a strict subset of DTBook and EPUB3.
http://nlbdev.github.io/nordic-epub3-dtbook-migrator/
GNU Lesser General Public License v2.1
8 stars 7 forks source link

Webpage update #511

Closed martinpub closed 2 years ago

martinpub commented 2 years ago

Hi @josteinaj!

This PR starts an update of the Nordic Migrator web page, preparing for an official release (following #396).

So far, I have updated the structure and gone through the About section, let me know what you think.

I suggest we leave this PR open until we are ready to publish (and I guess the GH web page is based directly off of the state of the gh-pages branch), so more commits will follow before it's ready for merging.

martinpub commented 2 years ago

Hi again @josteinaj, I added an update of the Resources section.

A question, should we make an updated list of validation rules for 2020-1 guidelines? I have a vague memory of us/Hugo starting on something like that early on in the guidelines update process? Noting that this is only for 2015-1 at the moment, sticking with minimally required updates at this point. But it would be nice to point to something similar for 2020-1 guidelines.

Next, i will turn my attention to testing the installation methods again from a user perspective, and see if there are any changes/clarifications to be done in that section.

josteinaj commented 2 years ago

Hi @martinpub. Looks great!

Yes, let's wait with merging until we release a new version.

Having an updated list of validation rules for 2020-1 would be useful, but maybe not required for making a release. It's good that you remembered it, I had totally forgot about it. There's a couple of 2020-XSLTs here thatwe can run: https://github.com/nlbdev/nordic-epub3-dtbook-migrator/tree/guidelines-revision/documentation

josteinaj commented 2 years ago

I've made a release now. Version 1.5.0:

The Docker image has not finished building yet, but unless something goes wrong, it will be available very soon: https://hub.docker.com/r/nlbdev/nordic-epub3-dtbook-migrator

martinpub commented 2 years ago

Great work @josteinaj!

Will continue with the docs update.

BTW, did you see my question in #396?

martinpub commented 2 years ago

@josteinaj I added the HTML documents generated from the Schematron XSLT. I think there is room for improvement when it comes to presentation, e.g. duplication of rule numbers, unclear distinction between "Global" and "Detailed" description, etc. Do you think it is useful as is? I'm not sure but I guess it doesn't hurt to add it. I hope it is not confusing to new users, but it might be ...

josteinaj commented 2 years ago

Hi @martinpub. I haven't had time to look at it yet but it doesn't hurt to add it, as you say, and we can improve it later if needed.

martinpub commented 2 years ago

Hi @martinpub. I haven't had time to look at it yet but it doesn't hurt to add it, as you say, and we can improve it later if needed.

OK, great. Logged this task separately in https://github.com/nlbdev/nordic-epub3-dtbook-migrator/issues/513, will leave it as is for now.

josteinaj commented 2 years ago

@martinpub can I merge this now or do you want to make more changes first?

martinpub commented 2 years ago

Hi @josteinaj, I have some minor edits left, and was still thinking about how best to phrase the Docker instructions, which will be most important.

martinpub commented 2 years ago

Hi again @josteinaj, made some additions now. What do you think about this? I have tried to strike a balance between a somewhat user-friendly explanation of the easiest way to run the migrator while not getting too deep into the specifics of various Docker setups.

Are there any updates to be made to the Pipeline 2 updater section based on the current release status?

Ping also @AndersEkl, do you see any obvious things we need to add/change?

If you both approve the changes, I guess they can be merged now. If we get questions later on, we can clarify and update the web page docs.

AndersEkl commented 2 years ago

Ping also @AndersEkl, do you see any obvious things we need to add/change?

@martinpub Where should I go in and read?

martinpub commented 2 years ago

@martinpub Where should I go in and read?

Yeah, sorry @AndersEkl, probably not the most straight-forward in a pending PR, but here is a link to index.html in the PR, where most of the changes are at.

josteinaj commented 2 years ago

I'm just gonna merge this one. We can easily adjust it later if needed (either directly or in a PR).

martinpub commented 2 years ago

I'm just gonna merge this one. We can easily adjust it later if needed (either directly or in a PR).

Thanks @josteinaj!