-
Notifications
You must be signed in to change notification settings - Fork 108
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
Worksheet reference #12
Comments
@jansiegel mentioned at the meeting that he had to remove underscore |
Correct, also, when the name of the sheet contains a |
I've changed format from Also, now I am not so sure, whether it's worth to pursue the support for other characters separating sheet name from the reference, because the tools supporting There are few more tasks which may be worth adding:
This looks like topics which could be covered by standard. Does ODDF says something about that? |
Closing the issue for now, as the things from the task are done, and my above comments can be extracted into different tasks. |
I haven't seen spec about keeping the TLDR; I think we should compare it as case-insensitive and display always as normalized to the name stored by the engine. Are helpers from #32 case-insensitive @swistak35 ? There is a fragment about escaping characters. Like you said, by a double single quote:
I will create a new task for this one 👍 The older draft for ODFF [1] explains why there is
And indeed the regex marks is as optional [1] https://www.oasis-open.org/committees/download.php/16826/openformula-spec-20060221.html |
No, they are not -- we should have either case insensitivity in all places or not :) I mean, I agree with you, but the change is not only scoped to these two helpers -- we also need to change unparser to return correct name, etc. -- can you create a task for that as well? For case insensitivity?
Sounds doable |
Current notation is
=$Sheet1.A1
, while the more popular is using exclamation mark=Sheet1!A1
and does support spaces='Sheet number two'!B4
We need:
The text was updated successfully, but these errors were encountered: