kimholmes

Results 9 comments of kimholmes

@carlos-logro @AlanGreene

@ncskier Just had a conversation with Carrie, the viz designer and she reminded me that we did decide to move those tabs down. The reasoning behind it was that, if...

View the high fidelity mockups at the following git links. Click on object to view associated specs: ### **PipelineRun Diagram View "Closed"** https://kimholmes.github.io/Exported-Specs/PipelineTopology_Closed/ ![PLRdiagram1](https://user-images.githubusercontent.com/21316329/67696702-b5553800-f97d-11e9-866b-97655e5f2ba9.png) ### **PipelineRun Diagram View "Open" -...

### **Hover states:** ![PLR_DiagramHover](https://user-images.githubusercontent.com/21316329/67704251-218a6880-f98b-11e9-8fb2-5872dcf2f9f6.png) ### **Node selection:** ![PLR_DiagramNodeSelection](https://user-images.githubusercontent.com/21316329/67704505-9e1d4700-f98b-11e9-91fe-09dc95282773.png) ### **Accordion selection:** ![PLR_DiagramAccordionSelection](https://user-images.githubusercontent.com/21316329/67704808-31567c80-f98c-11e9-8155-6831740cef77.png) ### **Toolbar and Cursor behavior:** ![PLR_DiagramToolbarCursor](https://user-images.githubusercontent.com/21316329/67705163-dd986300-f98c-11e9-9e08-82d77f3212e7.png) ### **View Logs:** ![PLR_DiagramViewLogs](https://user-images.githubusercontent.com/21316329/67705654-c7d76d80-f98d-11e9-9fba-588727ca93d6.png)

@AlanGreene If you're able to apply it only to the row being hovered on, that would be ideal.

@AlanGreene The only place we're surfacing the YAML in the designs is on the Service Accounts. But we can certainly add this capability. Happy to discuss.

@AlanGreene I'm interested to learn more about what you mean when you say that surfacing the YAML feels like cheating. Why was that approach supposed to be a short term...

@AlanGreene Ok. That makes sense. If we want to be consistent with what we're doing in other parts of the design for the UI, we'd add a YAML tab on...