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

Start formatting tables with horizontal scroll and fixed-width font #799

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

artbristol
Copy link
Contributor

@artbristol artbristol commented Dec 31, 2020

Followup from discussion on #74

In this approach the note's content is split into alternating table and text (i.e. non-table) content. The tables are then formatted with a fixed-width font, and horizontal scrolling is used to prevent text wrapping (wrapping makes tables basically unusable)

Before merging I'd at least need to address:

  • test breakage
  • adding more tests
  • consider what to do about editing tables
  • consider pushing down the parsing of table content into org-java library and storing result of the table/text parsing process in the database

@artbristol
Copy link
Contributor Author

artbristol commented Feb 16, 2021

@nevenz This works better than I expected. Do you think the ability to edit a table nicely is necessary or could we implement this display enhancement first?
Also this PR parses the tables on the UI thread which I thought would be bad, performance-wise, but it seems fine to me.

@artbristol
Copy link
Contributor Author

@nevenz I added a very bare-bones edit screen. I tried reusing the NoteViewModel but there is still a lot of duplication.
I was planning to add icons on the edit screen for actions like "reformat", "add/delete column", "add/delete row". What do you think?

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.

None yet

1 participant