So the only downside is that we risk to obfuscate history.
Trying it this change with a patch.
- Phabricator seems to understand that lines were moved.
- `hg annotate` shows all the moved lines as changed by the patch's changeset (all the `:root {…}` styles which are currently at the end).
- I suspect https://searchfox.org/mozilla-central/source/devtools/client/themes/variables.css uses `hg annotate`?
Thankfully the only affected lines are the ones for shared variables, and it's mostly Photon colors, so we don't hide the more relevant history. But still a trade-off to consider. Any thoughts?
Bug 1568786 Comment 2 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
So the only downside is that we risk to obfuscate history.
Trying out this change with a patch.
- Phabricator seems to understand that lines were moved.
- `hg annotate` shows all the moved lines as changed by the patch's changeset (all the `:root {…}` styles which are currently at the end).
- I suspect https://searchfox.org/mozilla-central/source/devtools/client/themes/variables.css uses `hg annotate`?
Thankfully the only affected lines are the ones for shared variables, and it's mostly Photon colors, so we don't hide the more relevant history. But still a trade-off to consider. Any thoughts?