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

Create definitions documentation #270

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kernelkind
Copy link
Member

Added docs/definitons.md to have a source of ground truth for Notedeck related definitions. @jb55 let me know your thoughts and if you have any revisions. This is going to help reduce the likelyhood of misunderstandings in the future

Signed-off-by: kernelkind <[email protected]>
@jb55
Copy link
Contributor

jb55 commented Aug 16, 2024 via email

@kernelkind
Copy link
Member Author

Not technically true, not sure if it will be true either. I think we
will stick with automatic horizontal sizes for the forseeable
future.

Can you elaborate on this? Which part is untrue? If you could provide a more accurate description that would be great.

@jb55
Copy link
Contributor

jb55 commented Aug 21, 2024

specifically :

Columns can be resized in the horizontal direction.

we don't have the technical ability to do that yet (manually at least)

@kernelkind
Copy link
Member Author

specifically :

Columns can be resized in the horizontal direction.

we don't have the technical ability to do that yet (manually at least)

My intention for the definitions.md is to have a shared understanding of what the app should be, what we're working towards, not what it currently is.

Do we want columns to be able to resized in the horizontal direction? I like the impl tweetdeck has, where you have discrete column widths Narrow, Medium and Wide. Is this something we want to do too?

@jb55
Copy link
Contributor

jb55 commented Aug 24, 2024 via email

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