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

Updated Tables Page #453

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

ReedGraff
Copy link
Contributor

I changed the table page to better match the descriptions which were laid out.

@ReedGraff
Copy link
Contributor Author

ReedGraff commented May 4, 2023

Is there anything else I should change on this one? @V1EngineeringInc

@V1EngineeringInc
Copy link
Owner

I like it. I have a couple small changes I want to make but I have been a bit wrapped up in a backend project. I will need a couple of days to get to this.

I really do appreciate it, thank you!

@jamincollins
Copy link
Contributor

I would love to see this published, I've been looking for ideas like this.

@jamincollins
Copy link
Contributor

Are there any additional details that can be provided around the table shown in the docs/img/MetalTubing.gif?

@V1EngineeringInc
Copy link
Owner

I have actually added a bit of info since this has been submitted.

I don't know anything about metaltubing.gif and is exactly why I have been hesitant to merge something like this. I am willing to put a link but if it is in the docs like this I feel the need to be fully supportive of it.

@ReedGraff
Copy link
Contributor Author

Are there any additional details that can be provided around the table shown in the docs/img/MetalTubing.gif?

Yes, there is, the table in the gif is a table I designed and built. The reason for it being able to swivel is that a full table wouldn't fit in my door unless I flipped it.

@ReedGraff
Copy link
Contributor Author

I have actually added a bit of info since this has been submitted.

I don't know anything about metaltubing.gif and is exactly why I have been hesitant to merge something like this. I am willing to put a link but if it is in the docs like this I feel the need to be fully supportive of it.

No worries, I removed the gif and replaced it with an image of one of the more supported designs. If you or @jamincollins were interested in learning more, however, feel free to email me about it here: [email protected]

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.

3 participants