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

Online editing problems and comments #547

Open
njarboe opened this issue Oct 25, 2021 · 6 comments
Open

Online editing problems and comments #547

njarboe opened this issue Oct 25, 2021 · 6 comments
Assignees

Comments

@njarboe
Copy link
Member

njarboe commented Oct 25, 2021

Checking out the new online editing system.

Important to add
Delete a row function
Label to indicate the tables can be edited

Other useful features to consider
Standard undo and redo of edits
Links to controlled vocab lists
Pulldowns for vocab lists when appropriate

@rminnett
Copy link
Member

Is this sufficient for deleting rows?

image

@rminnett
Copy link
Member

What about removing the Experiments counts and adding "Preview Data:" for published contributions and "Edit Data:" for private contributions:

image

@njarboe
Copy link
Member Author

njarboe commented Jan 17, 2022

I think these will work well and you should implement them.

I think not everyone will know to right click to add rows, columns, etc. I would add a line of text stating that. Maybe "Right click to add or delete columns and rows"

I also don't seem to be able to invoke the insert column option.

@njarboe
Copy link
Member Author

njarboe commented Jan 17, 2022

There is also excess whitespace around the Contribution name and tabs. This should be reduced to as little as possible. Same for the column headers and row data, if possible. Many people (especially students) mainly use laptops with small screens.
Screen Shot 2022-01-17 at 8 18 10 AM

@rminnett
Copy link
Member

I'll see what I can do about the whitespace. The insert column is intentionally disabled for now because it needs a mechanism to choose what the new column's field should be from the data model.

@rminnett
Copy link
Member

These changes are deployed, but I'll leave this issue open until the "Other useful features to consider" list is also deployed.

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

No branches or pull requests

2 participants