Conversation
Present default, component-based, and chromeless UI approaches within each Player SDK so developers can choose and navigate customization guidance without switching product areas.
|
|
I'm not a huge fan of this deeply nested navigation structure... 😕 Have a look at the breadcrumbs on https://docs-preview.optiview.dolby.com/pr-873/theoplayer/how-to-guides/web/ui/open-video-ui/guides/custom-ui/:
|
@MattiasBuelens , do you have a suggestion for how you prefer to handle this? |
I think I like that best. That puts it at the same level as "Connectors", which feels right. (There are probably more articles that deserve their own section outside of "How-to guides", but that's for another time.) |
|
On a more general note, we would like to highlight Open Video UI over the built-in UI, since the latter is pretty much frozen in time. (It's based on an old version of video.js, and there's no easy upgrade path without forcing customers to recheck all their CSS.) So perhaps we should put "Open Video UI" as the first subcategory of "UI" in the sidebar, and also put it at the top on the introduction pages? |
Agreed, we should move "Default UI" down to the end. |
|
We should also update the documentation page from the Open Video UI SDK to maybe move the following text to the |
|
Sidebars look good now! We should adjust the URLs too though: these articles are no longer in |
Since connectors live at |
|
I also noticed that if you're on Open Video UI for Web and you use the platform dropdown to switch to Android or React Native, then we just send you back to /theoplayer/android/ instead. Update the cross-platform linking logic to correctly handle UI pages. |
|
@FabienLavocat I'm still looking through, but Devin had this in the meantime:
It also called out this, but it seems less legitimate if we are ok just have breadcrumb items with no pages attached. It seems to be handled, since there's no link on those breadcrumb items, but I thought I'd mention if we do want the generated index pages for those breadcrumb items:
/open-video-ui/web/guides/ |
Summary
Test plan
Replaces #870 so the preview workflow can access repository secrets.