summaryrefslogtreecommitdiff
path: root/Userland/Applications/Spreadsheet/Cell.cpp
AgeCommit message (Collapse)Author
2021-12-16Spreadsheet: Handle emptying of cell containing only an '='RasmusNylander
Cell::set_data(String new_data) now checks whether the cell is a formula-cell and the new_data is an empty string. If this is case, it will no longer simply return and will now instead actually set the cell's contents to an empty string. This fixes an error whereupon committing the string "=" to a cell, it would not be possible to directly delete the cell's contents. Instead, it first had to be overwritten with another string, which then could be deleted. This could probably be done more elegantly. Right now, I believe, writing the string "=" to a (formula-)cell already containing an identical string will result in the cell being marked as dirty, even though nothing actually changed.
2021-12-12Spreadsheet: Avoid using Value.to_string_without_side_effects()Ali Mohammad Pur
We should use .to_string() and handle the possible exceptions. This makes the displayed cell contents so much more informative than '[object Object]' :^)
2021-11-11Everywhere: Pass AK::StringView by valueAndreas Kling
2021-06-17Spreadsheet: Remove 'return (...)' workaround in conditional formattingAli Mohammad Pur
Now that LibJS can evaluate expressions correctly, that workaround was breaking conditional formatting.
2021-04-22Everything: Move to SPDX license identifiers in all files.Brian Gianforcaro
SPDX License Identifiers are a more compact / standardized way of representing file license information. See: https://spdx.dev/resources/use/#identifiers This was done with the `ambr` search and replace tool. ambr --no-parent-ignore --key-from-file --rep-from-file key.txt rep.txt *
2021-02-23Everywhere: Rename ASSERT => VERIFYAndreas Kling
(...and ASSERT_NOT_REACHED => VERIFY_NOT_REACHED) Since all of these checks are done in release builds as well, let's rename them to VERIFY to prevent confusion, as everyone is used to assertions being compiled out in release. We can introduce a new ASSERT macro that is specifically for debug checks, but I'm doing this wholesale conversion first since we've accumulated thousands of these already, and it's not immediately obvious which ones are suitable for ASSERT.
2021-01-12Applications: Move to Userland/Applications/Andreas Kling