Repeated pressing of 7 in graphical EPG doesn't toggle item height cleanly

Issue #443 resolved
prl created an issue

Repeated presses of remote button 7 "No. of items toggle" doesn't toggle stably between two list lengths.

This is caused by not returning to the original list height as the basis of each toggle: instead it uses the current list height, which may have been affected by rounding of the item height.

Replication steps

In the graphical EPG PIG, set MENU>Number of rows to 8. Return to the EPG. Press 7 on the remote; the number of items and the item height doesn't toggle stably, the item height and number of items reduces at each step.

Comments (1)

  1. Peter Urbanec

    Fix bug #443: Repeated pressing of 7 in graphical EPG doesn't toggle item height cleanly

    [EPGList] in EPGList.setItemsPerPage(): Use the original list height to recalculate the itemheight instead of the current list height (which is often reduced by rounding down the list's itemheight).

    [EPGSelection] In EPGSelection.keyNumberGlobal():

    Use a logical expression instead of an if statement to toggle config.epgselection.graph_heightswitch.

    → <<cset e91dddd49a33>>

  2. Log in to comment