Improve datagrid to help the user read errors - Research
Opened this issue · 4 comments
Problem description
Right now, when the ODE previews the table, the app only indicates row numbers.
However, the error report will include the cell location to help the user identify errors. Therefore, we need to incorporate a table header, like the one Datawrapper, for example, has:
Is this possible?
I really like Datawrapper's approach to solving this problem. I'd use it as a starting point
Hello @romicolman , I have added the column headers and it seems to be working fine, so I think the conclusion is that it can be done 💯 ✨ and we can keep this one for the complete version. You can see in the PR #436 . This PR does not include updating the error reports, though. This PR is only adding the headers. Should we create another issue for updating the error reports to identify the cells by column letter and number, since this is only the research part?
Hey @romicolman I was just thinking of what happens when there are more than 26 columns ... My example only covers the letters of the alphabet.
Will the column after Z be 'A1' or 'AA' ? Do we have maximum amount of columns that the app supports?
Heyyy! Thanks for all the info.
Let's keep the same logic that we see in Google Sheets, Excel: A-Z and then 'AA', 'AB', 'AC', 'AD'. Since this is a research ticket, I'll create a new one and mention this issue.
Errors report: yes...we need to update that. We are working on it.