1553719 - Perfherder compare chooser should trim the revision id

Status: RESOLVED FIXED

(Tree Management :: Perfherder, enhancement, P1)

Description

[Paul Bone [:pbone]]

I like to copy-paste revision IDs into the perfherder compare chooser, I do not type them out in full. But sometimes copy paste copies a bit of whitespace on one or both ends of the revision id. If the perfherder compare chooser was able to trim this whitespace (internally, in its memory, don't update the control) then it would make it easier to use.

Comment 1

[Ionuț Goldan [:igoldan]]

Bug 1553734 will take care of the problematic selection.
Still, an enhancement like this would be useful, as that hash can come from sources other than Treeherder.

Comment 2

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/treeherder/pull/5228

Comment 3

[Ionuț Goldan [:igoldan]]

This got merged to master.

Comment 4

[Ionuț Goldan [:igoldan]]

Resolving this as it got deployed to production.