FlowMobile 8.0.8: menús compactos - #24
Conversation
- Cabecera de dos líneas y opciones en una sola en iPhone. - El panel deja la caja; muestra vídeos, audios y espacio en una línea. - Submenús con títulos cortos: Descargar, Lotes, Más, Actualizar.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4fc0f9f7ab
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| self.logo("PRUEBAS") | ||
| print(f"{YELLOW}Descargan archivos reales.{RESET}") | ||
| self.menu_item("1", "Rápida", "360p + M4A") | ||
| self.menu_item("2", "Completa", "5 plataformas") |
There was a problem hiding this comment.
Warn that the complete test starts up to 30 downloads
Selecting “Completa” and supplying all five URLs runs every target through the six cases returned by full_cases(), so this immediately starts up to 30 real media downloads without another confirmation. Describing the option only as “5 plataformas” removes the prior data-usage and download-count warning, which can cause substantial unexpected mobile-data and storage consumption.
Useful? React with 👍 / 👎.
| cli.logo("DESINSTALAR") | ||
| print(f"{YELLOW}Quita el comando flow y el código.{RESET}") | ||
| cli.menu_item("1", "Conservar datos") | ||
| cli.menu_item("2", "Borrar todo") |
There was a problem hiding this comment.
Name downloads in the destructive uninstall option
When this option is selected, uninstall(purge_all=True) removes the configured FlowMobile download, video, and audio directories in addition to application state. The shortened label and subsequent irreversible-action prompt no longer tell users that their downloaded media will be deleted, so someone interpreting “Borrar todo” as application data can permanently lose personal files; retain the explicit download/cookie/history scope in this confirmation flow.
Useful? React with 👍 / 👎.
| print( | ||
| f"{GRAY}{videos}v {format_bytes(video_size)} · " | ||
| f"{audios}a {format_bytes(audio_size)} · " | ||
| f"{format_bytes(free)} libre{RESET}" | ||
| ) | ||
| print(f"{tools_color}{tools_label}{RESET} {update_color}{update_label}{RESET}") |
There was a problem hiding this comment.
Preserve descriptive dashboard output in accessible mode
The dashboard now uses compact abbreviations such as 0v/0a and combines system and update statuses on one line even when interface_mode == "accessible". This removes the mode's previous descriptive labels and separately readable status lines while the rest of the CLI still provides accessible-specific clearing and progress behavior, making the main screen substantially less understandable to screen-reader users; keep a descriptive branch for accessible mode.
Useful? React with 👍 / 👎.
Publica FlowMobile 8.0.8 para que el iPhone reciba el menú compacto.
Tras fusionar hay que etiquetar `v8.0.8` para que la acción Release publique los assets.