Skip to content

topcat: add CellViewWindow#77

Open
fjebaker wants to merge 1 commit intoStarlink:masterfrom
fjebaker:master
Open

topcat: add CellViewWindow#77
fjebaker wants to merge 1 commit intoStarlink:masterfrom
fjebaker:master

Conversation

@fjebaker
Copy link
Contributor

@fjebaker fjebaker commented Mar 6, 2026

The "Cell View" is a new drop-down menu option when a user right-clicks on a cell in a TableViewerWindow. It pops open a small window that displays the full contents of that cell, with the idea being that it can be used to read the longer form textual descriptions that are stored in some tables.

Some screenshots:

Right clicking before selecting a cell shows a deactived menu item:

cell-view-deactivated

After selecting a cell, it is active and directly under the mouse for ease of access (right click and then immediately left click):

cell-view-activated

The cell view window itself:

image

@fjebaker
Copy link
Contributor Author

fjebaker commented Mar 6, 2026

Regarding a comment you made about drop downs being for columns only, I see how this conflicts with that. Is there a way you can put a divider in (as in <br>) on a drop down menu?

@mbtaylor
Copy link
Member

mbtaylor commented Mar 6, 2026

JPopupMenu.addSeparator()!

@fjebaker
Copy link
Contributor Author

fjebaker commented Mar 6, 2026

divider

How about it?

@mbtaylor
Copy link
Member

mbtaylor commented Mar 6, 2026

Sorry - as a newbie option it's going at the bottom not the top.

@fjebaker
Copy link
Contributor Author

fjebaker commented Mar 6, 2026

divider

I thought initially it would be more convenient at the top so that it's right under the mouse when you right click but after trying it out it's not bad at the bottom either. I'll update the PR.

The "Cell View" is a new drop-down menu option when a user right-clicks
on a cell in a TableViewerWindow. The Cell View is displayed at the
bottom after a divider, to make it distinct from column operations. When
clicked, it pops open a small window that displays the full contents of
that cell.
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