desktop/desktop

Support common mouse/keyboard combinations for zoom

Open

#3.913 geöffnet am 1. Feb. 2018

Auf GitHub ansehen
 (12 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (18.758 Stars) (9.945 Forks)batch import
enhancementhelp wantednot-planned

Beschreibung

Description

Please use common keyboard mouse combinations for zooming: Ctrl++ for zoom in Ctrl+- for zoom out Ctrl+0 for zoom reset

Issue 1: Shortcut description in the menu is wrong Issue 2: Numpad +/- is not supported as in Firefox, Chrome or IE Issue 3: Mouse scrolling is not supported, in contrast to almost all popular software titles

Version

GitHub Desktop version: 1.0.13 OS version: Windows 7, Windows 10

Steps to Reproduce (Issue 1)

Open the View menu

Expected behavior: it shows Ctrl++ for zoom in

Actual behavior:
Menu screenshot: grafik

Steps to Reproduce (Issue 2)

Press Ctrl+NumPad-+

Expected behavior: it zooms in as in Firefox, Chrome, or IE Actual behavior: nothing happens

Steps to Reproduce (Issue 3)

Hold Ctrl key and scroll with the mouse wheel

Expected behavior: it zooms in or out as in Firefox, Chrome, or IE Actual behavior: nothing happens

Contributor Guide

Tech Stack
typescriptreactelectron
Domain
desktop
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Electron accelerator knowledgeReact event handlingTypeScript basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
Review the current zoom implementation in the repository. Check the menu template file for incorrect accelerator strings. Examine how keyboard shortcuts are registered to include numpad events. Investigate how mouse wheel scroll with Ctrl is handled, possibly in the window event listeners. Consult existing comments in the issue for any suggested approaches.