1. 27 May, 2021 5 commits
  2. 25 May, 2021 2 commits
    • Salah-Eddin Shaban's avatar
      Freetype: use FT_Glyph_Stroke to create outlines · 074b4f3c
      Salah-Eddin Shaban authored and Salah-Eddin Shaban's avatar Salah-Eddin Shaban committed
      FT_Glyph_Stroke is more tolerant of font files that do not follow
      contour orientation conventions, like Helvetica Neue.
      It also creates unfilled outline glyphs that can be more efficiently
      blended.
      
      Fixes #21043
      074b4f3c
    • Steve Lhomme's avatar
      d3d11: remove Query based rendering wait · 141e07e1
      Steve Lhomme authored
      It is imprecise and on seek it may wait a lot of time as, while it's waiting,
      the decoder threads will flood new frames, delaying that much longer until the
      ID3D11DeviceContext is finished doing things. This workaround did more harm
      than good.
      
      Now that we have ID3D11Fence support (any recent Win10) the query based system
      should not be used. For older Windows using D3D11 we'll have less precise
      estimation of the time it took to actually render, meaning the frame drop will
      be less effective.
      141e07e1
  3. 24 May, 2021 2 commits
  4. 23 May, 2021 31 commits
    • Lyndon Brown's avatar
      qt: (hotkeys) improve translatability of strings · 0289edad
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      0289edad
    • Lyndon Brown's avatar
      core: ensure all hotkey options have longtext · 954ae268
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      with an earlier commit in this set having added a tooltip using longtext
      to the first column of the hotkey editing table, allowing users to get
      an additional description to help them understand the purpose of an
      action, it would be great for best UX if we have such a tooltip for
      each and every entry in the table. of the 113 entries, 9 were missing
      longtext and thus missing the tooltip.
      
      it was for this reason i skipped hotkeys in
      d75459a3.
      
      the "Select the hotkey to use..." form was used with that being fairly
      common for hotkey longtext.
      
      i used a single common longtext for the four zoom presets, as done
      already with bookmarks and subtitle text scaling, reducing translator
      burden.
      954ae268
    • Lyndon Brown's avatar
      core: move bookmark option text defines · 7d14aae8
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      these are not for hotkey options they are for playlist options
      7d14aae8
    • Lyndon Brown's avatar
      core: move a few misorganised hotkey declarations · 1cbb4d39
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      move the clear playlist and subtitle scaling hotkey options above
      the bookmarks and jump-size ones.
      
       - in the hotkey widget they are surely more interesting than the
         big block of bookmark ones that otherwise come at the end.
      
       - in help output, since the bookmark ones and jump sizes ones each
         declare sections, these options appear miscategorised as being a
         part of the bookmark section. this was overlooked in
         711733c6, or more likely perhaps
         lost from it in a rebase over time.
      1cbb4d39
    • Lyndon Brown's avatar
      33db5233
    • Lyndon Brown's avatar
      qt: (hotkeys) convert remaining loops to use iterators · 7da6f250
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      7da6f250
    • Lyndon Brown's avatar
      qt: (hotkeys) move param init · 2c124b48
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      2c124b48
    • Lyndon Brown's avatar
      qt: (hotkeys) remove unnecessary constructor param · 78b49ac0
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      the parent is always the table.
      78b49ac0
    • Lyndon Brown's avatar
      qt: (hotkeys) fix grid layout numbers · 4c38d200
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      we need 4 columns, not five. one element on the line with four was being
      pointlessly stretched across two.
      4c38d200
    • Lyndon Brown's avatar
      qt: (hotkeys) simplify and optimise global population · 001574dc
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      now the above non-global check has been reduced to a single name check,
      we can strip the reverse of that here with an else.
      
      the one remaining condition is just to avoid work for empty mappings.
      001574dc
    • Lyndon Brown's avatar
      qt: (hotkeys) do not skip item on null shorttext (action label) · a3e2e9d0
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      unlikely and better to add it rather than skip. being there with a blank
      label would better help diagnose what went wrong in testing, or if it
      slipped past testing, someone is more likely to notice and report the
      problem leading to a fix.
      
      note that we must conditionally use `qfut()` because the underlying
      translation function does not like nulls, and my fix for that was rejected.
      a3e2e9d0
    • Lyndon Brown's avatar
      qt: (hotkeys) remove redundant problem check · ba06a165
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      we now rely upon checking option name rather than shorttext. it is somewhat
      less likely that a mistake with duplicate option names will have been made
      compared to duplicate shorttext, and duplicate option names presents an
      even bigger and broader problem that we currently do nothing about. this is
      not very useful towards helping with that, we'd want a full check of the
      entire table's option names if we cared to properly check things here.
      ba06a165
    • Lyndon Brown's avatar
      qt: (hotkeys) use key name for global data population · a758d770
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      this avoids doing a translation lookup of the displayed shorttext, and
      using the option name just feels more "correct".
      
      note that we strip the "global-" prefix with `+7` to get the match.
      a758d770
    • Lyndon Brown's avatar
      qt: (hotkeys) rework global hotkey table population · 0c423eeb
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      rather than getting a list of matching items, instead iterate over the list
      and compare, thus avoiding the allocations in building that list. this also
      puts us in a position to locate the row by option name rather than
      shorttext, which might be better.
      0c423eeb
    • Lyndon Brown's avatar
      qt: (hotkeys) fix overlooked old use of "Unset" for an unset item situation · e7abd3e2
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      the "Unset" text used to be used for all unset cases, but a long time ago
      all except this one got changed to use an empty string instead. having
      spent a bit of time trying to track down an answer in the git history as to
      the question of why, i gave up. i found no clear answer in the time spent
      looking. it very well may have been simply overlooked rather than
      deliberately left to create a clear distinction that a failure occurred.
      
      i'm going to assume that it was overlooked. i've thus changed it to return
      an empty string. i don't expect this to cause any problems for the two
      different uses in the Qt code.
      e7abd3e2
    • Lyndon Brown's avatar
      qt: (hotkeys) improve/update table docs · 1342e605
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      1342e605
    • Lyndon Brown's avatar
      qt: (hotkeys) fix missing translation of initial displayed global assignment · da95e7ad
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      the displayed text is always displayed translated for non-global, and this
      is the case for edits for global, but the initial unedited displayed text
      was not translated.
      
      due to the way the global data is temporarily stored in a `QMap`, it seems
      that the easiest way to fix this is to simply convert from `QString` to
      raw string pointer with `qtu()` and then back to `QString` with translation
      applied with `qfut()`.
      da95e7ad
    • Lyndon Brown's avatar
      10128a9f
    • Lyndon Brown's avatar
      qt: (hotkeys) clarify table label · d6c283a5
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      let's start the text with something describing the actual purpose
      of the overall control, rather than leaping directly into
      instructions on how to use it. this may not be so important for
      where the table is shown in the simple view, but makes a notable
      difference to use in the advanced view, where it looks very odd
      without it.
      
      also, let's clarify those instructions a bit.
      d6c283a5
    • Lyndon Brown's avatar
      qt: (hotkeys) add a tiny amount of padding to row height · 62c8efd7
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      a tiny touch of padding makes a world of difference to the look of
      the table imo.
      
      notes:
      
       - originally i had specified `-1` for width, signalling default,
         which worked perfectly. however now when testing before publishing
         this work, i found the change was no longer working. however it
         does work when specifying `0` for width instead. it seems
         behaviour changed or a bug was introduced in newer Qt versions.
      
       - i tried an alternative solution of using `sizeHint()` on the
         first element to get the initial dimentions, then increasing the
         height by 12px before applying that, however this does not work
         since values of `-1` are returned from `sizeHint()`. there does
         not seem to be a way of getting the actual size to increase it, or
         to set an amount of margin/padding to apply.
      62c8efd7
    • Lyndon Brown's avatar
      qt: (hotkeys) add longtext tooltip to first column of each row · ddd09846
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      this makes longtext of hotkey actions available to users, helping them to
      understand the purpose of the actions listed in the table.
      
      this does not interfere at all with the tooltip giving instructions on
      how to perform editing, that still appears when hovering over the other
      two columns. no tooltip existed for the first column previously.
      ddd09846
    • Lyndon Brown's avatar
      qt: (hotkeys) simplify and optimise table filter · 81e22cde
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      the old solution:
       1) collected a list of all matching items. this could include duplicates
          when searching all columns.
       2) iterated over all of the items, for each one checking whether or not it
          was contained within that set of matches.
      
      the new solution simply iterates over the items, checking the one or more
      columns and setting the hidden property as appropriate as it does so.
      81e22cde
    • Lyndon Brown's avatar
      qt: (hotkeys) update table filtering upon column selection change · 4bfa2900
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      i.e. update when user changes field selection combobox.
      
      the filter function is changed here to fetch the filter text directly
      from the textbox to support this.
      
      note that this uses Qt's `connect()` directly, bypassing the vlc `CONNECT()`
      macro. doing otherwise, it compiles but does not work.
      4bfa2900
    • Lyndon Brown's avatar
      qt: (hotkeys) use comma+space as visible multi-binding separator · 8a9a91e9
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      keep the existing separator (tab) for stored values, but swap with
      comma+space separator for display. it works much better this way imo and
      is much less confusing. it also happens to match the style used by the
      `QKeySequenceEdit` control.
      
      we thus avoid the problems that the default multi-key bindings create, as
      is the default on Linux currently, whereby with play/pause for example you
      see "Space       Media Play Pause" which obviously looks odd. now you will
      see "Space, Media Play Pause".
      
      if users assign the actual comma key in a multi-key binding, it should
      not really be a problem for them to grasp what's what. for instance
      ",, Media Play Pause".
      8a9a91e9
    • Lyndon Brown's avatar
      qt: (hotkeys) resize 'hotkey' table column to contents · adb88154
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      the use of multi-key bindings (default) on some platforms, like mine,
      makes the small starting width of the column problematic. this helps
      avoid users having to resize the column manually to see everything.
      adb88154
    • Lyndon Brown's avatar
      qt: (hotkeys) fix multi-key conflict checking · 4000ae13
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      the hotkey conflict check did not understand actions which have more than
      one key/key-combination (tab separated) assigned, and as such it allowed
      keys/key-combinations to be assigned to more than one action in some cases.
      
      e.g. on Linux systems, the "key-play-pause" action has both "Space" and
      "Media Play Pause" keys assigned, but because the string is
      "Space\tMedia Play Pause" not "Space" and because the conflict check just
      matches against the entire string, the conflict is overlooked if trying to
      set "Space" on something else.
      
      conflict checking became broken by default for non-MacOS systems back in
      v2.1 due to baf3e50b introducing default
      multi-key bindings, without testing the hotkey editor properly. on Windows,
      thanks to e8668774, it happened to become
      unbroken by default for v2.2.
      
      note that the editor currently does not support users assigning multi-key
      bindings; so the problem only comes up when multiple bindings are the
      default, or if users force them by manually editing their config file.
      
      note also that this obviously only fixes Qt; the problem likely still
      exists in the MacOS code (which I leave for a mac guy to handle), but
      thankfully there are no such dual bindings there by default, so users
      won't notice (unless they've manually modified their config file).
      4000ae13
    • Lyndon Brown's avatar
      qt: (hotkeys) fix non-bold wheel event name · 068f5d0d
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      in the 'selected' text, inconsistent with that done for keyboard events.
      068f5d0d
    • Lyndon Brown's avatar
      qt: (hotkeys) avoid re-doing key conversions · 2de662ef
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      we had:
       1. event making translated copy for 'selected' text.
       2. conflict check making an untranslated copy (one commit back in this
          set it was another translated copy).
       3. the 'accepted' code, if triggered, making a further one of each.
      
      a single copy of each can be created and stored in public attributes by
      the event handlers, and used in all of these places, without any negative
      impact. this replaces the need for the public `keyValue` attribute, and
      replaces the need to give the conflict checker a copy of the integer.
      2de662ef
    • Lyndon Brown's avatar
      qt: (hotkeys) avoid pointless translation in conflict check · c86f0e52
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      with the previous commits having put us in the position to do this, let's
      adjust the conflict checker to avoid comparing translated text, which is
      entirely unnecessary.
      
      condition split into two for neatness due to line length issue.
      c86f0e52
    • Lyndon Brown's avatar
      qt: (hotkeys) rework conflict check · 656a7a26
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      this replaces fetching a list of conflicts to react to, with
      iterating over the table and looking for conflicts as we do so.
      
      the primary purpose of this is not efficiency, though it is more
      efficient; it is actually to support the 'multi-key assignment conflict
      check' fix to come in a subsequent commit in this set.
      
      furthermore it is needed to allow improvement of the conflict checker
      regarding avoiding translation (we need to be able to compare the custom
      data attributes for that, not the displayed text, as `findItems()` does).
      656a7a26
    • Lyndon Brown's avatar
      qt: (hotkeys) do not show warning for conflict with self · c2b0d66d
      Lyndon Brown authored and Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf committed
      if you tried to set a hotkey to use the same key as it already was set to,
      it would detect a conflict in that the selected key was already in use,
      failing to recognise though that it was conflicting with itself, thus
      resulting in a pointless and silly warning.
      c2b0d66d