Account for Dalamud's maximum CellPadding in Auto-Scrolling

Infiziert90 / ChatTwo

European Union Public License 1.2

17 stars 13 forks source link

Account for Dalamud's maximum CellPadding in Auto-Scrolling #86

Closed zbee closed 1 month ago

zbee commented 1 month ago

Account for Dalamud's maximum CellPadding very simplistically as ImGui.GetScrollMaxY() apparently does not account for this variable but ImGui.GetScrollY() seems to, and these two are compared to Auto-Scroll. Because of this apparent discrepancy auto-scrolling does not work on CellPadding values greater than 4 in custom Dalamud Styles.

16 because 4 is the default Dalamud CellPadding (which works), and 20 is the max.

Fixes #85.

Infiziert90 commented 1 month ago

Implemented a dynamic calculation for it, still thanks for the PR 58a6567f5a72d812d841d0326ebc161ef663475a