openvolley / ovscout2

An R Shiny app for scouting volleyball data files
https://ovscout2.openvolley.org
Other
5 stars 1 forks source link

Change display of scout options #17

Closed mchiapello closed 2 years ago

mchiapello commented 2 years ago

Hello, is it possible to increase the video loop by decreasing the button size in pop-up interface? I feel the video loop is super useful, but a bit small.

Thank you

raymondben commented 2 years ago

Yes, good suggestion, thanks. Will make it an option (which will also be useful in accommodating varying screen sizes).

raymondben commented 2 years ago

Make font size (on buttons) variable too, rather than relying on scaling the view in the browser.

raymondben commented 2 years ago

Implemented in dev branch, you can now start the scouter with something like

ov_scouter(app_styling = list(review_pane_width = 40))

(review pane becomes 40% of screen width and the buttons will shrink to accommodate it).

mchiapello commented 2 years ago

Hi Ben, the new option works very well! Thank you for implemented it.

Just a small side effect: the Botton names remain of the same font size, so some of them are not readable or part of the name is hidden behind the next button.

image

This is very low priority issue.

mchiapello commented 2 years ago

Is possible to not have the review panel on top of the court panel? This can be especially useful when you also scout the opponent team and you do not know the player or you cannot see the shirt number. Having the ability to see the court panel can be extremely useful.

This is what I called court panel: image

Here for example, I cannot see the number of the receiving player. If the court panel is not hidden by the review panel I can probably infer it.

image

raymondben commented 2 years ago

If you press 'z' at this point, the popup and review pane will be hidden (while you hold 'z' down) and you can see the court panel

raymondben commented 2 years ago

(By the way, if the keyboard shortcuts keys do not suit you, you can change them. The defaults are set by ov_default_shortcuts(), so you could:

my_shortcuts <- ov_default_shortcuts()
## modiify that
ov_scouter(..., shortcuts = my_shortcuts)
mchiapello commented 2 years ago

I did not know the existence of this keyboard shortcut. Thank you