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

Tables #5

Merged
merged 2 commits into from
Feb 29, 2020
Merged

Tables #5

merged 2 commits into from
Feb 29, 2020

Conversation

YoomamaFTW
Copy link
Collaborator

@YoomamaFTW YoomamaFTW commented Feb 29, 2020

  • Toolbar for table added
  • Style for table added
  • Functionality for adding table added

TODO

  • Need to grab user's current row/column for appending
  • Sample needs to show how to use this properly using menu items

* Toolbar for table added
* Style for table added
* Functionality for adding table added
@YoomamaFTW
Copy link
Collaborator Author

When it comes to getting the index, try row.rowIndex

* User can add or delete current row
* User can add or delete current column
* If you want to add a row or column to the left, then add your own JS that replicates mine... since they're really similar.
@YoomamaFTW
Copy link
Collaborator Author

This commit allows for appending and deleting items. If you want to add a row to the left instead of the right (same for column) you can use rowIndex - 1 or columnIndex - 1 and duplicate the functions and add your swift hook accordingly.

@YoomamaFTW
Copy link
Collaborator Author

For the menu items, you can look up how to add menu items yourself. Plenty of tutorials. The added functions are simple to understand: isCursorInTable (returns bool), addRowToTable, deleteRowFromTable, addColumnToTable, deleteColumnFromTable.

The isCursorInTable can help with menu functionality (like Google Docs!).

@YoomamaFTW YoomamaFTW merged commit 08665da into master Feb 29, 2020
@Andrew-Chen-Wang Andrew-Chen-Wang deleted the tables branch July 10, 2021 17:33
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.

1 participant