Hi , I am removing you from the tester assignee field on these because they need to get tested asap, as they are going out in the Q2 release and you are on vacation. You can still do PO acceptance when you get back.
Rasmus Wølk March 8, 2018 at 6:26 PM
FYI: The option for adding AppIcon into a PaneHeader is now merged into stripes-components.
This is how it works: <Pane appIcon={{ app: 'instances', key: 'items' }} />....
Rasmus Wølk March 7, 2018 at 1:24 PM
Edited
It's closer to the blueprint. We'll find some good solution to the minor issues.
I have submitted a pull request for the option of putting app icons into PaneHeader's so once this is merged in you can simply pass a "appIcon" -prop (and if needed: appIconKey for a specific icon) to the Pane or PaneHeader which will render an AppIcon in the PaneHeader correctly. I'll let you know once that's merged in
Niels Erik Nielsen March 7, 2018 at 1:06 PM
Of course, cool, now we have:
And the instance details:
The fonts are not quite like in the blue print, but at least the icons now fit the font size better.
Rasmus Wølk March 6, 2018 at 2:04 PM
You can change the size of the icon by adding the prop: size and setting it to "small".
This will make the icon smaller but we might need to adjust it a bit further to make it work properly. Let me know how it works out with changing the size.
Purpose: Implement latest UX changes from Filip - according to the discussion Niels and Filip had on Slack this evening:
Scenarios
1. Scenario
When using the Inventory App, and
viewing a given instance record,
then when clicking the 'View Holdings' button
the detailed view of the holding record will open up as an overlay.
Note: the pane header saying Holdings (...) - see attached png. - will not be a link
2. Scenario
When viewing the detailed holdings record
the headline is "Holdings data" - see the updated UX screen as of 1/9/2018