Low priority. But given that ChopsUI is now a thing, I think it makes sense to roll the Infra CRDX button into ChopsUI.
We could expose it as a Polymer component (or just raw web component), which would make the code easier to read than the copypasteable snippet we currently have: https://chromium.googlesource.com/infra/infra/+/master/crdx/feedback/
Or, if we want to just move the feedback button to live alongside the ChopsUI elements, we could keep it as is and just make sure the button is at leats documented in ChopsUI. Now that https://chopsui-gallery.appspot.com/ can expose arbitrary mardkwon files as documentation pages, I think it'd be nice to consolidate various frontend miscellaniary into one location if nothing else.
Low priority. But given that ChopsUI is now a thing, I think it makes sense to roll the Infra CRDX button into ChopsUI.
We could expose it as a Polymer component (or just raw web component), which would make the code easier to read than the copypasteable snippet we currently have: https://chromium.googlesource.com/infra/infra/+/master/crdx/feedback/
Or, if we want to just move the feedback button to live alongside the ChopsUI elements, we could keep it as is and just make sure the button is at leats documented in ChopsUI. Now that https://chopsui-gallery.appspot.com/ can expose arbitrary mardkwon files as documentation pages, I think it'd be nice to consolidate various frontend miscellaniary into one location if nothing else.
Comment 1 by zhangtiff@chromium.org
, Apr 18 2018