ux(button): show the label as a tooltip when QueryaButton truncates it
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Área
- accessibility, frontend
Línea de trabajo
Empieza en querya_button.dart, en la disposición de la etiqueta y la lógica de los tooltips mencionadas alrededor de las líneas 375 y 513. Usa LayoutBuilder y TextPainter para comprobar si la línea única supera el ancho disponible de la etiqueta, conservando los tooltips explícitos y omitiendo los botones que solo tienen icono. Añade o actualiza pruebas para una etiqueta truncada, una etiqueta que cabe y los tooltips explícitos; el trabajo estará terminado cuando solo la etiqueta truncada reciba su texto completo como tooltip.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
From a review of the component library, checked against dev.
Problem
QueryaButton draws its label with maxLines: 1 and TextOverflow.ellipsis (querya_button.dart, ~line 375). The tooltip is only shown when tooltip is set or the button is icon-only (~line 513). A label cut short by a narrow toolbar or a long translation therefore cannot be read by hovering.
Not this
Do not simply fall back to label as the tooltip: every button would then show "Save" over "Save" on hover.
Change
- Show the label as the tooltip only when it is actually truncated: lay the text out with a
TextPainter(maxLines: 1, the button's text style, the width the label gets) and checkdidExceedMaxLines, in aLayoutBuilder. An explicittooltipstill wins. - Check that a button which fits shows no tooltip, and that one which does not shows the full label.
Acceptance
- Narrow
QueryaButtonwith a long label shows the full label as a tooltip on hover. - A button that fits shows no tooltip of its own.
- The extra layout pass is not paid by buttons with an explicit tooltip, or by icon-only buttons.
Considered in the same review and left out
AppToastkeeps its overlay in statics (app_toast.dart:_entry,_overlay). The review wanted anAppToastScopefor multi-window use; the app has one window (bitsdojo_windowonly restyles it), so there is no problem to solve today.QueryaTabStripannouncing a tab change: each tab is aSemantics(button, selected)with its ownFocus, and arrow keys move focus, so a screen reader reads the selected tab already. An extraSemanticsService.announcewould read it twice.
- Lenguaje dominante
- Dart
- Estrellas
- 23
- Forks
- 1
- Merge medio
- 32 min
- PR fusionados (30 d)
- 537
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de QueryaHub/Querya-Desktop
-
ui(dialogs): make destructive query and DML preview dialogs adaptive and prevent table name overflowAbiertobug frontend
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
QueryaHub/Querya-Desktop#1642 ·
Los mantenedores suelen responder en 1 día
-
bug error-handling settings
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
QueryaHub/Querya-Desktop#1637 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
QueryaHub/Querya-Desktop#1636 ·
Los mantenedores suelen responder en 1 día
-
enhancement marketplace
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
QueryaHub/Querya-Desktop#1635 ·
Los mantenedores suelen responder en 1 día
-
enhancement P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
QueryaHub/Querya-Desktop#1595 ·
Los mantenedores suelen responder en 1 día
Todos los issues de QueryaHub/Querya-Desktop
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
jhomlala/betterplayer#1609 ·
Los mantenedores suelen responder en 1 día
-
[FR] 缓存键放置在外面Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
bggRGjQaUbCoE/PiliPlus#3310 ·
Los mantenedores suelen responder en 1 día
-
package:code_builder type-bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
type:feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
[material_ui] CalendarDatePicker day numbers are cut off with large text scale on narrow screensAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día