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

Minor issues with table view #333

Closed
6 of 7 tasks
jonorthwash opened this issue Nov 25, 2018 · 17 comments · May be fixed by #337
Closed
6 of 7 tasks

Minor issues with table view #333

jonorthwash opened this issue Nov 25, 2018 · 17 comments · May be fixed by #337

Comments

@jonorthwash
Copy link
Owner

jonorthwash commented Nov 25, 2018

  • Once the table view table has been resized, it doesn't resize automatically when the window size is adjusted. If it's desired to keep the window a particular width, there could be a "lock size" feature, but in general the default should be to resize it on window resize even if it's previously been manually resized.
  • Text in column headers (including the "«" characters) can't be clicked to collapse/expand the column. Text shouldn't block the collapse/expand feature.
  • When editing the contents of a cell, pressing Esc should probably keep the cell highlighted, but leave edit mode. Currently it leaves table navigation entirely.
  • There should be a keyboard shortcut for entering table view which should also enter table navigation mode (even if table view is already enabled).
  • Currently table navigation mode allows you to go into cells of collapsed columns, but does not highlight those cells. It should highlight those cells (probably in a slightly different way than uncollapsed cells).
  • Related to the above, there should be a keyboard shortcut for expanding and collapsing the column of the currently highlighted cell.
  • Delete/backspace should clear the contents of a cell.
@diogoscf
Copy link
Contributor

Any ideas on what the mentioned shortcuts could be?

@jonorthwash
Copy link
Owner Author

For table mode, something with a t maybe? I don't know what the general schema is, and there's a lot to think about in terms of conflicts: browser and OS shortcuts, entering text in the graph view, etc.

For expanding and collapsing the column, I'm not sure. Do you have any thoughts on this?

@diogoscf
Copy link
Contributor

diogoscf commented Dec 1, 2018

I’ll think about possible shortcuts.

  • Currently table navigation mode allows you to go into cells of collapsed columns, but does not highlight those cells. It should highlight those cells (probably in a slightly different way than uncollapsed cells).

Actually it does highlight the collapsed cells for me

@jonorthwash
Copy link
Owner Author

Actually it does highlight the collapsed cells for me

Aha, it works in chromium for me. So this is a firefox-specific bug I guess?

@jonorthwash
Copy link
Owner Author

The collapsing issues still seem to be present in firefox too—things are working a lot more nicely in chromium in general.

@diogoscf
Copy link
Contributor

diogoscf commented Dec 2, 2018

Aha, it works in chromium for me. So this is a firefox-specific bug I guess?

I tested it in both. That’s really weird

@jonorthwash
Copy link
Owner Author

What version of firefox? I'm running 60.0, it seems.

@diogoscf
Copy link
Contributor

diogoscf commented Dec 3, 2018

That issue was actually happening when the table got rebuilt (when rebuild() was called) but it happened in chrome and firefox for me...
Just in case the new commit doesn't fix the problem, I'm running 63.0.3

@jonorthwash
Copy link
Owner Author

So shift-- seems to do the trick?

@jonorthwash
Copy link
Owner Author

Or just -.

@jonorthwash
Copy link
Owner Author

Currently if you enter text into an empty grid cell in a collapsed, that text disappears into the grid cell (probably rightly so). Then if you try to edit it (by pressing enter), it doesn't show the text in the entry box, and the entered text seems to get concatenated to the existing text. Also, a couple times I got it to display text in a collapsed row, but I'm having trouble reproducing it.

Also, if zoom of the graph is played with (especially ctrl-0), the ability to collapse and uncollapse columns is lost until table view is deactivated and reactivated.

What's the keyboard shortcut for entering table view and table navigation?

@diogoscf
Copy link
Contributor

diogoscf commented Dec 5, 2018

ctrl-alt-t for entering table view and table navigation. I’ll check the other issues later

@diogoscf
Copy link
Contributor

diogoscf commented Dec 5, 2018

So which should the behaviour be?

  1. If you try to enter text into a collapsed cell, nothing happens
  2. If you try to edit a collapsed cell, the existing text appears so you can change it based on that

@jonorthwash
Copy link
Owner Author

Probably something like 2, except with the column expanding:
3. If you try to edit a collapsed cell, the column uncollapses and let's you edit normally.

@jonorthwash
Copy link
Owner Author

The shortcut ctrl-alt-t doesn't work for me; it's a window-manager-level shortcut.

@diogoscf
Copy link
Contributor

Ok. It's now alt-n

@jonorthwash
Copy link
Owner Author

Closing this issue; remaining issues to #499.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants