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

Added A simple index #212

Closed
wants to merge 3 commits into from
Closed

Added A simple index #212

wants to merge 3 commits into from

Conversation

Ank-22
Copy link
Contributor

@Ank-22 Ank-22 commented Nov 5, 2024

@gridhead
I realize that it may not be essential, but I believed it was a useful and straightforward addition that could assist new users or visitors to the project.

Please let me know if you would like any changes.

I kept the index simple, but I can create a more detailed one if needed.

I like the idea of using emojis with the section headers; if you'd prefer, I can add them to all sections or remove them from the index.

@gridhead gridhead requested review from gridhead and sdglitched and removed request for gridhead November 6, 2024 03:27
Copy link
Owner

@gridhead gridhead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the impromptu pull request.

Please make the following requested changes.

README.md Outdated
@@ -66,6 +66,20 @@
the development members</b>.
</p>



## 📚 Index
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The emojis might not be available on platforms like PyPI etc.

Suggested change
## 📚 Index
## Index

Comment on lines 68 to +70



Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please keep a maximum inter-section space of one newline.

Comment on lines +79 to +82




Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please keep a maximum inter-section space of one newline.

Comment on lines +73 to +78
1. **[Screenshots](#screenshots)**
2. **[Download](#download)**
3. **[Usage](#usage)**
4. **[Development](#development)**
5. **[Contribution](#contribution)**
6. **[Disclaimer](#disclaimer)**
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does seem to work on GitHub but not on PyPI.

@gridhead gridhead self-assigned this Nov 6, 2024
@gridhead gridhead added the documentation Improvements or additions to documentation label Nov 6, 2024
@gridhead
Copy link
Owner

gridhead commented Nov 6, 2024

I do not see much use in this if it does not work on PyPI, as GitHub already seems to have a dedicated semantic outline feature for project documentation.

{56491442-E368-4A1F-BE19-7419AF948BFA}

@Ank-22 Ank-22 mentioned this pull request Nov 6, 2024
@Ank-22
Copy link
Contributor Author

Ank-22 commented Nov 6, 2024

Please continue this discussion on pull request #213, Due to some technical error

@Ank-22 Ank-22 closed this Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants