rstudio/rstudio

RTools download progress dialog isn't focused

Offen

#11.521 geöffnet am 28.06.2022

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (1.060 Forks)batch import
bugelectrongood first issuewindows

Repository-Metriken

Stars
 (4.384 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 14h 15m) (202 gemergte PRs in 30 T)

Beschreibung

Steps to reproduce

  • On Windows without RTools installed, create a C/C++ file
  • When prompted to install RTools, select yes

Observe that the download dialog appears but under the main window. If you do not follow through with the install and repeat the steps above, it may show the dialog on top.

For a user with the main window maximized, the download dialog would be hidden and would likely never be noticed. It should appear on top of the main window.

See SessionInstallRtools.cpp for how the download works. It executes R code in the session to initiate the download to a temporary directory. https://github.com/rstudio/rstudio/blob/bb265d0dbedece7ced0325526ba14cb13684cb2d/src/cpp/session/modules/build/SessionInstallRtools.cpp#L126-L130

Contributor Guide

Research-Richtung
Untersuchen Sie, wie der Download Dialog erstellt wird, und stellen Sie sicher, dass er als modal oder als untergeordnetes Fenster des Hauptfensters festgelegt wird, damit er oben erscheint.
Tech Stack
cppr
Domain
desktop
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++QtWindows UI
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55