nfdi4plants / Swate

Excel Add-In for annotation of experimental data and computational workflows.
https://swate-alpha.nfdi4plants.org
MIT License
31 stars 6 forks source link

[Feature Request] Rename table to template name on import #489

Closed shiltemann closed 1 month ago

shiltemann commented 1 month ago

Is your feature request related to a problem? Please describe. I often load a template, then manually rename the table from the default title "NewTableX" to the name of the template.

Describe the solution you'd like It would be convenient if it automatically renamed the table to the template name when loading a table from template.

Describe alternatives you've considered Manually renaming is not a big deal, but I just find myself using the template name as table title most of the time, so it feels like it could be automated and save people some time.

Freymaurer commented 1 month ago

Doing things the user does not specifically intend to do is always hit-or-miss. Imagine you added a template and did not want to rename your table. Then you wold be very annoyed because you wanted to add a template and not rename the table.

In the reverse case of you wanting to rename the table, you also do not get what you want but you get what you would expect.

We could in the future offer an additional button/checkbox to rename the table.

shiltemann commented 1 month ago

Fair enough, perhaps only if the table name is still at the default? Offering a checkbox makes it the most explicit in terms of expectations, agreed.

Or perhaps instead of having it be a 2 step process of adding a new table followed by loading a template, make it a single action, "add table from template"?

Freymaurer commented 1 month ago

Yeah the template add logic will soon receive a rework, partially based on how protocol import from files looks like:

image

Freymaurer commented 1 month ago

Tracked in https://github.com/nfdi4plants/Swate/issues/499 will close here