Status Update
Comments
km...@keepersecurity.com <km...@keepersecurity.com> #2
More suggestions:
-
Allow to override containerElevation for RichTooltipBox - RichTooltipColors has container param, but doesn't allow overriding containerElevation (which mixes color with theme primary color, resulting in undesired color)
-
Allow to set or suggest the position of shown tooltip.
-
When isPersistent=true, allow dismissing by clicking outside and still passing through the click event
For example, when some persistent tooltip is shown for a button in screen's top area, user should be able to click something else in screen without having to click twice (once to dismiss the tooltip, second time to actually dismiss the tooltip)
-
Allow to customize PlainTooltipMaxWidth/RichTooltipMaxWidth
-
Accessibility - Hard to close isPersistent=true tooltip in TalkBack mode by clicking outside (this is probably an issue with PopupWindow itself)
-
Accessibility - TalkBack says "Popup window" when tooltip is shown (might be expected)
-
M3 spec doesn't seem to allow it, but tooltip arrows would be also useful. If not added to spec, maybe expose some currently internal stuff in M4 Tooltip implementation that will allow developers to implement arrows without having to rewrite it all?
si...@google.com <si...@google.com>
si...@google.com <si...@google.com>
ma...@google.com <ma...@google.com>
ap...@google.com <ap...@google.com> #3
It sounds like the autohide behavior you are looking for is similar to the isPersistent parameter for tooltip state:
When this is set to true, the tooltip should be able to remain on the screen until the user clicks outside of it. The request for a modifiable timeout seems reasonable to me, wdyt Kevin? Something to defer to design perhaps?
pr...@google.com <pr...@google.com> #4
@isPersistent = false
to rememberTooltipState()
. These changes might not be available yet in the current release, but it should be included in the next one. Thanks!
Plain tooltip should be used for quick descriptions, if you have a lot of text that requires longer than 1.5 seconds to read please use rich tooltip and make it actionable.
@
- I think this is a reasonable ask, will file a bug to address.
- We recently merged a change to tooltips that exposes the
PopupPositionProvider
, you should be able to alter the actual position with this parameter. - You can use the
focusable
parameter to achieve this flow. - Dallas What do you think about exposing the maxWidth for Plain and Rich tooltips? From my understanding, this isn't something that we normally expose.
- Yeah this is part of the
Popup
implementation. If you'd like to escalate you can file an accessibility bug with the wider compose team. - Yup, this is expected.
- This is something that we're currently working on! No exact ETA, but it would be in M3.
Description
Jetpack Compose component(s) used: NavigationBarItem
Android Studio Build: 2022.1.1 Canary 9
Kotlin version: 1.7.0
Steps to Reproduce or Code Sample to Reproduce:
1. Use NavigationBar with NavigationBarItem
2. Use overridden colors with NavigationBarItem with a transparent/translucent indicatorColor
3. The NavigationBarItem composable overrides the alpha channel of the color such that only solid indicator colors can be used.
It would be really helpful if we could use a transparent/translucent indicatorColor. We'd like to hide the pill color entirely to match approved design. The pill was heavily opposed and so we can't have it visible. But we can't hide the pill without completely copying the NavigationBarItem composable and A LOT of supported code, just to remove the `.copy(alpha = animationProgress) part of the indicator composable.
Can this Composable be modified to either (1) support transparent/translucent indicatorColor values and/or (2) completely hide the indicator composable (pill shape on the selected item).
Stack trace (if applicable):
```
<add stack trace here>
```