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

Update smol-slime.md: Added information on what required to build smol slimes and list of community projects #321

Open
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

Depact
Copy link

@Depact Depact commented Feb 1, 2025

Added information on what required to build smol slimes and list of community projects

Added information on what required to build smol slimes and list of community projects
Copy link
Contributor

@ShineBrightMeow ShineBrightMeow left a comment

Choose a reason for hiding this comment

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

Some information seems redundant. As this is under the DIY section, some of this should be basic knowledge already or the visitor should be familiar from building ESP trackers. Just add to existing sections if needed.

The Case/PCB section can go lower down the page, giving the visitor a chance to get an introduction of the hardware, software, and firmware first.

This seems to be a copy and paste from Philip6294's message on Discord. As for Aed's case, the coin cell battery is not specified as the other batteries are. What about Lyall's case?

@Depact
Copy link
Author

Depact commented Feb 2, 2025

@ShineBrightMeow thanks for the review! I pushed my review-based changes.

  • The DIY section was deleted until I had a good way to return it.
  • Changed community projects list to table and added information about some components where I found it specified. My change was so similar to Philip6294 because I thought it was a greatly structured message that sadly wasn't on wiki.
  • Added LyallUlric case.

@ShineBrightMeow
Copy link
Contributor

ShineBrightMeow commented Feb 8, 2025

I think "Community Cases/PCBs" sounds better since it is the same project/tracker.

As for the USB column, I think it is a bit redundant since Lyall and Scawanf's build/case both have USB making every item Yes.

Please add "Edited by 'Your Username'." at the bottom credits.

Copy link
Contributor

@ShineBrightMeow ShineBrightMeow left a comment

Choose a reason for hiding this comment

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

Everything else looks accurate other than some formatting changes.

src/diy/smol-slime.md Outdated Show resolved Hide resolved

**SlimeVR Discord:** <a href="https://discord.gg/SlimeVR" target="_blank">https://discord.gg/SlimeVR</a>
### Community Firmware
Copy link
Contributor

Choose a reason for hiding this comment

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

I am not sure if padding out the tables, so they are all the same width would look better.

image

- LIS3MDL (Not Tested)
- MMC5983MA

### Push Button/Momentary Switch
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe the Button, Switch, and Battery can be in a single table format too. It would reduce the vertical spacing between headers.

image

Copy link
Author

Choose a reason for hiding this comment

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

I'm not sure that I'm getting a vision how it should look. So I'm not sure how to improve. From what I see, those parts are too distinct to be in one table.

Copy link
Contributor

Choose a reason for hiding this comment

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

I am thinking maybe another header called "Other Components" and then you have the a table with Component/Description (or Purpose) and you can list the Button, Switch, and Battery. What do you think?

Copy link
Author

@Depact Depact Feb 9, 2025

Choose a reason for hiding this comment

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

It makes some sense since currently, those chapters are tiny, but I'd keep them separated to avoid any possible confusion. Those parts are a small part of the list so your idea should work. I'm just trying to not confuse people.

src/diy/smol-slime.md Outdated Show resolved Hide resolved
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 this pull request may close these issues.

2 participants