Update default template with UG links and useful information#2225
Update default template with UG links and useful information#2225lhw-1 merged 22 commits intoMarkBind:masterfrom
Conversation
jovyntls
left a comment
There was a problem hiding this comment.
Template mostly LGTM! I'm not a fan of the empty pagenav on the right (it seems a bit weird to have a pagenav that has a "Chapters of this page" header but no contents) - could we add some unobtrusive headers to populate the pagenav? I think it's still worth showcasing that the pagenav exists since it's one of the key features of the default template.
Co-authored-by: Jovyn Tan <61113575+jovyntls@users.noreply.github.com>
That's a good point - I'll include more sections from the main MarkBind page to make the site longer (and thus have functional page nav) 👍 |
|
@lhw-1 maybe set up a site, deploy it, and add the link to the description? Easier to preview that way? |
tlylt
left a comment
There was a problem hiding this comment.
Thanks @ lhw-1, this revamp is great. Some nits as mentioned.
As much as possible, I think we should use MarkBind features on this page to present the appropriate information. Not suggesting that we go back to the current site where we include different versions of the panel state to showcase the style of the header when expanded or minimized, but use appropriate components to display information. Panels, box, tooltip, etc are all possible.
I'll look at the main page and other relevant sections once more, and see what we can add to the template without bloating it too much. Ideally, it should be long enough to make the page navigation a useful addition (currently it's too short for the page nav to be useful), while making use of a good number of MarkBind components, but not so much to the point of bloating it. |
I have updated the template to include more content (to make our pageNav more useful & to show off some of our components)! You can check out the new version at the deployed site here. |
tlylt
left a comment
There was a problem hiding this comment.
Some nits, otherwise LGTM for me
yucheng11122017
left a comment
There was a problem hiding this comment.
Thanks for the work @lhw-1! This seems to be a much better landing page than the current 👍
Some minor nits on my end!
jovyntls
left a comment
There was a problem hiding this comment.
LGTM, really minor nits!
Some smaller thoughts:
- "Placeholder Topic Pages" is a bit of a strange title for the page nav. I'm thinking "On this page" might be better?
- The "What did you just do?" header sounds a bit negative to me, a bit like "What have you done?!?!" 😅 I think this is really minor so I'm fine with leaving it as it is. Other suggestions: "What's going on?", or no header at all
For the first point, @tlylt pointed it out earlier, so I changed it to "Topics"! As for the second point, that's a good point - I've changed it to "What just happened?" 😅 |
|
Hi @lhw-1, I'm getting some errors when trying to use the page nav at the side. It doesn't move to the correct header which is really odd... Not sure if its something wrong with the deployment or is it a local thing as well? |
@lhw-1 can you try building and deploy a version with the latest web built. It could be due to the recent updates to anchor scroll. |
I've just tried deploying the latest site and locally serving it, and the page nav is not working on either. But I've also tried switching to the Edit: My web was not built to the latest 😅 |
Did you |
Ok, I've built the latest web and re-deployed - the page nav for the deployment now seems to work 😅 |
yucheng11122017
left a comment
There was a problem hiding this comment.
LGTM! Thanks a lot @lhw-1
What is the purpose of this pull request?
Overview of changes:
Addresses a few of the main outcomes of the discussions in #2214, and updates the
defaulttemplate to include links to the main sections of the User Guide. Also possibly addresses #519.Here is a preview of what it currently looks like:
I've kept the new look to be as simple as possible, and have modelled it based partially on the MarkBind User Guide - Authoring Contents section and partially on the Gatsby initial site.
This is just a possible direction we could take the default template towards - if there are any suggestions to improve this current look, or if there are any concerns / issues with this, please highlight as needed!
Anything you'd like to highlight/discuss:
We should keep in mind to keep the template as simple as possible while including the main features that a user using
markbind init --convert --template defaultwould want to have in their site. (E.g. siteNav and pageNav are essential for this - any other suggestions are welcome.)Testing instructions:
Run
markbind initin a new directory to initialize a site built using thedefaulttemplate.Alternatively, you can view the deployed version of the template here.
Proposed commit message: (wrap lines at 72 characters)
Checklist: ☑️