Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zoom Limiter für sehr kurze Projekte #1

Closed
rstockm opened this issue Feb 25, 2018 · 13 comments
Closed

Zoom Limiter für sehr kurze Projekte #1

rstockm opened this issue Feb 25, 2018 · 13 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@rstockm
Copy link
Member

rstockm commented Feb 25, 2018

der zoom Limiter produziert derzeit noch ein Problem bei neuen Aufnahmen. Vorgehen: ich starte ein leeres Projekt, der Zoom ist recht weit draußen. Ich starte die Aufnahme, der Aufnahmebalken kriecht nur wenig von der Stelle. Ich zoome rein um zu schauen, dass da wirklich alles seine Ordnung hat - und kann nicht mehr zurück zoomen auf eine mittlere Stufe, da das Projekt eben noch sehr kurz ist.
Man kann natürlich mit alt+zoom das umgehen, besser wäre es aber, eine sinnvolle Untergrenze zu definieren unter der der Zoomlimiter nicht zuschlägt. Sprich - man kann immer bis 5 Minuten rauszoomen oder so was.

Standardmäßig startet eine Aufnahme mit einer Ansicht von 3 Minuten, mindestens bis dahin sollte man also immer heraus-zoomen können.

@rstockm rstockm created this issue from a note in Ultraschall 5 (Ideen) Feb 25, 2018
@rstockm rstockm added the bug Something isn't working label Feb 25, 2018
@rstockm rstockm added this to the 3.1.1 milestone Feb 25, 2018
@mespotine mespotine moved this from Offen / Ideen to In Arbeit in Ultraschall 5 Mar 4, 2018
@mespotine
Copy link
Contributor

Der Fix temporär aus den alten Issues hier reingepackt, bis ichs ins Git einpflegen kann...
zoomlimiter_fix.zip

Minimum Anzeige ist nun 180 Sekunden.

@rstockm rstockm moved this from In Arbeit to Offen / Ideen in Ultraschall 5 Mar 5, 2018
@mespotine
Copy link
Contributor

Pfeiltaste runter hat ja gar keinen Zoomlimiter. Soll das so und wenn ja, warum?

@mespotine mespotine moved this from Offen / Ideen to In Arbeit (Feature Branch) in Ultraschall 5 Mar 11, 2018
@mespotine mespotine moved this from In Arbeit (Feature Branch) to Test und Review (Pull Request) in Ultraschall 5 Mar 11, 2018
@rstockm
Copy link
Member Author

rstockm commented Mar 11, 2018

@mespotine ich habe die Pfeiltaste nach unten hinzugefügt, das macht schon mehr Sinn. Problem aber: Dein Fix funktioniert gut für die Tastatur, mit der der Pinch-to-Zoom Geste auf dem Trackpad jedoch ist es unverändert so, dass der Zoomlimiter zu früh greift.

@mespotine
Copy link
Contributor

Kann ich leider nicht hacken, da ich kein Trackpad habe....
Aber im Prinzip müsste die eine Zeile (mit GetProjectLength und <180 drin) sich auch in das andere Skript einbauen lassen können...

@mespotine
Copy link
Contributor

Ich hatte da mal versucht was zu hacken, wenn ich mich erinnere, aber das gab lauter Probleme. Hatte es aber nur mit MouseWheel testen können und nicht mit Trackpad...

@rstockm
Copy link
Member Author

rstockm commented Mar 11, 2018

Öhm. Wieso sind das zwei Skripte und nicht eines? Wo ist denn das zweite?

@mespotine
Copy link
Contributor

ultraschall_zoomlimiter.lua das sowohl zoom in als auch zoom out macht für pinch.

ultraschall_zoom_out_limiter.lua macht nur zoom out(und ist das, was ich gehackt habe)

Die ähnlichen Namen sind etwas unglücklich, aber mir fiel da auch nie was besseres ein...

@mespotine
Copy link
Contributor

Reaper macht bei MultiZoom leider keinen Unterschied zwischen rein und rauszoomen, zumindest nicht von den Keybindings her, daher müssen wir in einem Script Beides machen.
Im Gegensatz zu den normalen Shortcut-tasten, wo wir flexibler sind und die Usecases sauber voneinander trennen können...

@rstockm
Copy link
Member Author

rstockm commented Mar 11, 2018

Hm okay. Ich schaue mir das nochmal genauer an.

@rstockm
Copy link
Member Author

rstockm commented Mar 11, 2018

So, ich habe das jetzt auch für das Trackpad eingestellt: scheint gut zu laufen. Kann das abschließend jetzt nochmal wer mit einem Mausrad testen?

@mespotine
Copy link
Contributor

Mach ich morgen nachmittag denn...

@mespotine
Copy link
Contributor

mespotine commented Mar 12, 2018

Pfeiltaste runter hatte kurioserweise immer noch keinen Zoomlimiter. Habs nun ergänzt und sollte(TM) jetzt funzen.

Mousewheel (und auch Pfeiltaste runter/-) haben aber während REC ein Problem. Sie limitieren zwar, aber forcieren den Playcursor in die Mitte.
Da dieser am Ende des Projekts ist, heißt das soviel wie, dass das gesamte Projekt bis zur Mitte des Arrangeview angezeigt wird, wobei dieses "Mitte" durchaus weiter rechts, als auch weiter links sein kann.
Das war auch der Grund, weshalb ich das Hacken daran aufgegeben hatte. Ich hatte einfach nicht herausbekommen, woran das liegt.
Vermutlich irgendwo ein Setting für während Rec, das da Einfluss hat.

mousewheel-zoomlimiter-problem

rstockm added a commit that referenced this issue Mar 12, 2018
@rstockm rstockm moved this from Test und Review (Pull Request) to Done (Merge in Master) in Ultraschall 5 Mar 12, 2018
@rstockm
Copy link
Member Author

rstockm commented Mar 12, 2018

Das beschriebene Verhalten würde ich als das Gewünschte bezeichnen.
Ich habe noch den Pfeil-Rauf mit aufgenommen, der zentriert jetzt genauso auf den Edit-Cursor.
Ich musste nochmal einen neuen Branch für die Änderungen anlegen, da der alte nicht mehr sinnvoll zu mergen war: zu weit vom Master entfernt in bezug auf die keyb.ini :-(

@heikopanjas heikopanjas modified the milestones: 3.1.1, 4.0 Dec 12, 2019
@mespotine mespotine moved this from Done (Merge in Master und Docs) to Done for Good in Ultraschall 5 Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Ultraschall 5
  
Done for Good(4.0 und früher)
Development

No branches or pull requests

3 participants