Add options to change seekbar's step/jump size It has been fixed to 1s/10s. Ideal seekbar step size depends on personal preferences, so I made it customizable. I changed my config to 5s/20s after little fiddling. https://codeberg.org/pocka/plac-for-gtk4/issues/25

Changes

4 changed files (+60/-2)

• data/jp.pocka.plac.gtk-adwaita.gschema.xml
  • @@ -58,6 +58,16 @@ SPDX-License-Identifier: Apache-2.0
    <summary>Show stop button</summary> <description>Show stop button next to play/pause button.</description> </key> + <key name="seek-step-size" type="u"> + <default>1</default> + <summary>Seek seconds</summary> + <description>Number of seconds to seek backwards/forwards when an arrow key is pressed.</description> + </key> + <key name="seek-page-size" type="u"> + <default>10</default> + <summary>Jump seconds</summary> + <description>Number of seconds to seek backwards/forwards when PageUp, PageDown, or Ctrl + arrow key is pressed.</description> + </key> <key name="show-browse-item-separators" type="b"> <default>false</default> <summary>Show separators for browse items</summary>
• src/Application.vala
  • @@ -129,8 +129,54 @@ namespace PlacGtkAdwaita {
    focus_browse_items_on_load.title = "Focus browse items on load"; focus_browse_items_on_load.subtitle = "Move focus to the first browse item or search entry right after the page is loaded."; + var seek_step_size = new Adw.EntryRow(); + seek_step_size.title = "Seekbar step size"; + seek_step_size.input_purpose = NUMBER; + settings.settings.bind_with_mapping(Settings.SEEK_STEP_SIZE, seek_step_size, "text", DEFAULT, (to, from) => { + var size = from.get_uint32(); + to.set_string(size.to_string()); + return true; + }, (from, variant) => { + var text = from.get_string(); + var raw_val = uint.parse(text, 10); + if (raw_val == 0) { + // Vala's type definition wrongly defines return type signature of + // "SettingsBindSetMappingShared" as non-null, despite it being nullable and + // even documentation saying "return null in case of an error." + // This cast suppress compiler warning. + return (Variant) null; + } + + return new Variant("u", raw_val); + }, null, null); + seek_step_size.add_suffix(new Gtk.Label("secs.")); + + var seek_page_size = new Adw.EntryRow(); + seek_page_size.title = "Seekbar jump size"; + seek_page_size.input_purpose = NUMBER; + settings.settings.bind_with_mapping(Settings.SEEK_PAGE_SIZE, seek_page_size, "text", DEFAULT, (to, from) => { + var size = from.get_uint32(); + to.set_string(size.to_string()); + return true; + }, (from, variant) => { + var text = from.get_string(); + var raw_val = uint.parse(text, 10); + if (raw_val == 0) { + // Vala's type definition wrongly defines return type signature of + // "SettingsBindSetMappingShared" as non-null, despite it being nullable and + // even documentation saying "return null in case of an error." + // This cast suppress compiler warning. + return (Variant) null; + } + + return new Variant("u", raw_val); + }, null, null); + seek_page_size.add_suffix(new Gtk.Label("secs.")); + var keyboard_navigation = new Adw.PreferencesGroup(); keyboard_navigation.add(focus_browse_items_on_load); + keyboard_navigation.add(seek_step_size); + keyboard_navigation.add(seek_page_size); keyboard_navigation.title = "Keyboard Navigation"; var fixed_discovery_udp_port_enabled = new Adw.SwitchRow();
• src/Settings.vala
  • @@ -25,6 +25,8 @@ namespace PlacGtkAdwaita {
    public const string LABEL_PARSING_ENABLED = "label-parsing-enabled"; public const string SHOW_SEEK_BY_10SECS = "show-seek-by-10secs"; public const string SHOW_STOP_BUTTON = "show-stop-button"; + public const string SEEK_STEP_SIZE = "seek-step-size"; + public const string SEEK_PAGE_SIZE = "seek-page-size"; public const string SHOW_BROWSE_ITEM_SEPARATORS = "show-browse-item-separators"; public const string FOCUS_BROWSE_ITEMS_ON_LOAD = "focus-browse-items-on-load"; public const string DISABLE_CSD = "disable-csd";
• src/Widgets/PlaybackToolbar.vala
  • @@ -257,8 +257,8 @@ namespace PlacGtkAdwaita {
    schedule_seek((int64) next_value); }); - seek.set_increments(1, 10); - + settings.settings.bind(Settings.SEEK_STEP_SIZE, seek.adjustment, "step-increment", GET); + settings.settings.bind(Settings.SEEK_PAGE_SIZE, seek.adjustment, "page-increment", GET); settings.settings.bind(Settings.SHOW_SEEK_BY_10SECS, seek_backwards_10, "visible", GET); settings.settings.bind(Settings.SHOW_SEEK_BY_10SECS, seek_forward_10, "visible", GET); settings.settings.bind(Settings.SHOW_STOP_BUTTON, stop, "visible", GET);