infragram icon indicating copy to clipboard operation
infragram copied to clipboard

table of contents bar to make navigation easier

Open shubhangi013 opened this issue 2 years ago • 18 comments

To enhance both user and developer experience, we can have a table of contents for the readme that can make navigating through the various headings easier.

one can take reference from: https://github.com/publiclab/plots2#table-of-contents This is a beginner-friendly issue. You'd need to know a bit about the syntax followed by GitHub. one can read about it here: https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax

shubhangi013 avatar Apr 04 '22 10:04 shubhangi013

@TildaDares @jywarren If this fits in as a FTO, you can label it so that other contributors can partake. :smile:

shubhangi013 avatar Apr 04 '22 10:04 shubhangi013

@shubhangi013 Is the collapsible menu meant for Infragram or the README. If it's for Infragram, where would the menu be and what content would it contain? Thanks!

TildaDares avatar Apr 04 '22 14:04 TildaDares

@shubhangi013 Is the collapsible menu meant for Infragram or the README. If it's for Infragram, where would the menu be and what content would it contain? Thanks!

It is for the Readme

shubhangi013 avatar Apr 04 '22 14:04 shubhangi013

But GitHub already has a Table of Contents menu.

TildaDares avatar Apr 04 '22 14:04 TildaDares

Oh, I meant it in a sense that we can have a table of contents fro the readme with headings like 'Contributing' and 'First time?' so that it's easier for developers and users to navigate through the instructions of the readme. Just like the reference I've linked. It has a table of contents.

shubhangi013 avatar Apr 04 '22 14:04 shubhangi013

Is this the Table of Contents you're talking about?

Screenshot 2022-04-04 at 15 14 36

TildaDares avatar Apr 04 '22 14:04 TildaDares

Yes!

https://user-images.githubusercontent.com/78342516/161563391-8e5bd3f4-b891-4d30-8b11-6f881557b2c4.mp4

It makes it easy to navigate

shubhangi013 avatar Apr 04 '22 14:04 shubhangi013

I like where the idea is coming from but wouldn't a second Table of Contents be redundant. Also, Github's ToC is easier to navigate because the user will always see it as they scroll.

TildaDares avatar Apr 04 '22 14:04 TildaDares

Right. makes a lot of sense :sweat_smile: Didn't think about that. I came across this readme template and thought it would be a great addition to the current readme. but yes, it would be redundant. I'll close this issue. Thankyou!

shubhangi013 avatar Apr 04 '22 14:04 shubhangi013

On second thought @shubhangi013, I think we can add one. I checked other Public Lab repos and they have a second Table of Contents. If you want, you can reopen this and create an FTO for it.

TildaDares avatar Apr 04 '22 14:04 TildaDares

But instead of the ToC being a dropdown, can it be like the one in the plots2 repo? https://github.com/publiclab/plots2#table-of-contents

TildaDares avatar Apr 04 '22 14:04 TildaDares

Yes it would be better. GitHub already has a collapsible one. I will reopen this!

shubhangi013 avatar Apr 04 '22 14:04 shubhangi013

Also, as mentioned in https://github.com/publiclab/plots2/issues/10153, there should be a template while I open an issue that is FTO. I don't think we have the bot here in infragram repository. Should it be installed or would you label this as an FTO? I've edited it to make it more clear

shubhangi013 avatar Apr 04 '22 14:04 shubhangi013

@shubhangi013 Wonderful observation. I'll take a look at that. Thank you!

TildaDares avatar Apr 04 '22 14:04 TildaDares

Thankyou! :heart:

shubhangi013 avatar Apr 04 '22 14:04 shubhangi013

Hi @shubhangi013 thank you for the ideas!

If we make a copy of the existing first-timers issue template (for the bot) which is at https://github.com/publiclab/infragram/blob/main/.github/first-timers-issue-template.md

And we put it in the corresponding folder as we have in the plots2 repository: https://github.com/publiclab/plots2/tree/main/.github/ISSUE_TEMPLATE/

We should be able to use it as a template here. It's a really good idea.

Note that the version in the ISSUE_TEMPLATE folder has some extra material at the top - both a table, and a short explanation on how to customize the template:

---
name: "\U00002728 First Timers Only issue"
about: Open a First Timers Only issue for someone
title: ''
labels: fto-candidate
assignees: ''

---

### 💡Format this Issue Into a First Timers Issue
This has been marked as a good candidate for becoming a [first-timers-only issue](https://code.publiclab.org/#r=all) like these, meaning that it's **simple, self-contained**, and with some extra formatting✍️, could be a great entry point for a new contributor. If you're familiar enough with this code, please consider **reformatting it as a first-timers-only issue**, and then ping @publiclab/reviewers to get it labelled🏷. Or, if this is not your first time, **try to solve it yourself**🧑‍💻!

****

We'd have to copy that in as well. Would you be interested in opening a pull request for that?

jywarren avatar Apr 05 '22 19:04 jywarren

Also for background, the TOC at plots2 was created before GitHub added the TOC feature. But i'm not strongly for or against adding another TOC!

jywarren avatar Apr 05 '22 22:04 jywarren

Hi @shubhangi013 thank you for the ideas!

If we make a copy of the existing first-timers issue template (for the bot) which is at https://github.com/publiclab/infragram/blob/main/.github/first-timers-issue-template.md

And we put it in the corresponding folder as we have in the plots2 repository: https://github.com/publiclab/plots2/tree/main/.github/ISSUE_TEMPLATE/

We should be able to use it as a template here. It's a really good idea.

Note that the version in the ISSUE_TEMPLATE folder has some extra material at the top - both a table, and a short explanation on how to customize the template:

---
name: "\U00002728 First Timers Only issue"
about: Open a First Timers Only issue for someone
title: ''
labels: fto-candidate
assignees: ''

---

### 💡Format this Issue Into a First Timers Issue
This has been marked as a good candidate for becoming a [first-timers-only issue](https://code.publiclab.org/#r=all) like these, meaning that it's **simple, self-contained**, and with some extra formatting✍️, could be a great entry point for a new contributor. If you're familiar enough with this code, please consider **reformatting it as a first-timers-only issue**, and then ping @publiclab/reviewers to get it labelled🏷. Or, if this is not your first time, **try to solve it yourself**🧑‍💻!

****

We'd have to copy that in as well. Would you be interested in opening a pull request for that?

Yeah, I'd love to! Thanks!

shubhangi013 avatar Apr 06 '22 03:04 shubhangi013