Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unused TopicToc.md #46

Closed
damithc opened this issue Jun 6, 2018 · 6 comments · Fixed by #78
Closed

Remove unused TopicToc.md #46

damithc opened this issue Jun 6, 2018 · 6 comments · Fixed by #78

Comments

@damithc
Copy link
Contributor

damithc commented Jun 6, 2018

TopicToc.md at lower level topics don't seem to be used

@jamessspanggg
Copy link
Contributor

When you mean lower level topics, do you mean the following?
Screen Shot 2020-04-05 at 2 19 00 AM

@damithc
Copy link
Contributor Author

damithc commented Apr 5, 2020

It refers to topicToc.md files. It is used only for the following popover, and those are there for the top level topics.
image

In fact, I don't mind getting rid of those popovers as well all topicToc.md files can be removed with it.

@damithc
Copy link
Contributor Author

damithc commented Apr 5, 2020

The preview.md is another file that can be removed.

@jamessspanggg
Copy link
Contributor

So basically,

  • all popovers in all topicToc.md should be removed.
  • all preview.md should be removed as popovers no longer used.

As for removing unused topicToc.md, may I get an example from which topic it should be removed? I can't find any that is unused tho.

@damithc
Copy link
Contributor Author

damithc commented Apr 5, 2020

As for removing unused topicToc.md, may I get an example from which topic it should be removed? I can't find any that is unused tho.

We can remove all those files. [root]/index.md is the only place they are used. Instead of including topicToc.md in that page, we can just put the actual content (which is just one bullet point anyway).

@jamessspanggg
Copy link
Contributor

We can remove all those files. [root]/index.md is the only place they are used. Instead of including topicToc.md in that page, we can just put the actual content (which is just one bullet point anyway).

Ok noted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants