Repository navigation
fix(macos): la ventana de sede queda delante del escritorio, no deja iconos en el Dock y Spotlight abre el escritorio con una sede abierta - #2057
Merged
Conversation
…o en el Dock mientras espera, y Spotlight abre el escritorio con una sede viva
…critorio desde Spotlight
…ftHSM de Homebrew en macOS
… un llavero aparte de macOS
…a configuración raíz
sgomez
commented
Oct 20, 2026
sgomez
left a comment
Owner
Author
There was a problem hiding this comment.
CLEAN
Alcance completo (primera revisión), origin/main...a6556f4e. La PR no cierra ningún issue: sin spec, la fidelidad se mide contra la descripción de la PR.
Checks aún en marcha en a6556f4e: Plataformas / macOS, Plataformas / Windows, Imagen nativa, Cadena Rust, Cadena TypeScript, Analyze (rust). No se han corrido en local.
[fix]
event_loop.rs:28: el proceso de sede lanza el escritorio en cadaReopen, también con la ventana de sede a la vista. Pinchar en el Dock el icono de una sede que está firmando abre el escritorio y le cede el primer plano, que tapa la ventana de sede, justo el problema que arregla esta PR. Con la ventana de sede visible, elReopendebería traerla al frente (show_the_site_window); el escritorio solo se lanza si está oculta.
[note]
testdata/keychain/certs.sh:openssl pkcs12 -legacyno existe en el LibreSSL de/usr/bin/openssl, así que la receta necesita elopensslde Homebrew en elPATH. Además, si una importación falla a medias, el llavero queda creado y el siguienteinstallresponde «ya existe» sin completarlo.Serving::from_the_start()ya no solo dice «ventana arriba desde el principio»: ahora también marca el rol de sede (reopens_the_desktop). El nombre se queda corto.- El pegamento de AppKit (
yield_the_foreground_to,launch_the_desktop_on_reopen, Dock) no tiene pruebas automáticas; es razonable, porque solo se puede probar a mano en macOS, y la PR documenta esas pruebas manuales.
| if !desktop::adapters::process::launch_the_delivered_urls(app, &event, window_is_up) { | ||
| serving.only_for_links.store(true, Ordering::SeqCst); | ||
| } | ||
| if serving.reopens_the_desktop |
Owner
Author
There was a problem hiding this comment.
[fix] Con la ventana de sede visible, un Reopen (clic en el Dock) lanza igualmente el escritorio y le cede el primer plano, que tapa la sede a mitad de trámite. Lanza el escritorio solo si la ventana de sede está oculta; si está a la vista, tráela al frente con show_the_site_window.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Qué cambia
En macOS, al firmar en una sede:
afirma://y lo traía al frente, y la ventana de sede se quedaba detrás. Ahora el escritorio cede el primer plano al proceso de sede (yieldActivationToApplication).service), pero al ocultar su ventana el proceso de sede sale del Dock y vuelve a entrar al mostrarla. Firmar varios documentos ya no deja un icono por proceso.Para desarrollar en macOS:
just certs instally las pruebas de Rust que usan el token encuentran el SoftHSM de Homebrew (tests/support/softhsm.rs, dado de alta enAUTHORISED_SITES).just keychain-certs install|uninstallmete los.p12detestdata/fnmten un llavero aparte, el que ve la app instalada, y lo borra entero al quitarlo..gitignoreexcluye.claude/worktrees/: biome no lee el gitignore global y se encontraba varias configuraciones raíz.Pruebas
just lint-rustyjust structural-guardsen macOS.pkcs11_token,remembered_certificateysite_token_signingen verde.installed_signingsigue con dos rojos que ya lo estaban antes (el Almacén NSS de rFirma en macOS).just keychain-certsinstalado, quitado y reinstalado: los 11.p12entran y la lista de búsqueda queda como estaba.packaging/flatpak/sources.lockse actualizó a mano con elsha256deCargo.lock(sinuven local);Cargo.locksolo añadelibca las dependencias deobjc2-app-kit.