Hi,

I would like to report a formatting issue in the Service List file, located at:

/usr/lib/enigma2/python/Components/ServiceList.py

Issue:

Within the _buildOptionEntryAddTimeDisplay function, the time display template is defined as:

textTpl = "00:00 - 00:000"

However, the time is actually formatted using:

"%02d:%02d - %02d:%02d" % (beginTime[3], beginTime[4], endTime[3], endTime[4])

This means both the start and end times should be displayed as two-digit values (HH:MM). The extra digit in the end time template ("00:000") is therefore incorrect.

Impact:

• The incorrect template may result in inaccurate width calculations.
• It might also lead to inconsistent UI appearance.

Suggested fix:

Change the template to:

textTpl = "00:00 - 00:00"

This change will ensure that the display width is correctly calculated based on the actual time format.

Best regards.

Thank you for your comment! I agree with you that the purpose of textTpl is to calculate the required width for the addtimedisplay text correctly, and the idea of using a string to determine the maximum width makes sense.

However, I think "00:00 - 00:000" might not be the best choice here, as it doesn’t fully match the actual format of addtimedisplay, which is "HH:MM - HH:MM". Using "00:00 - 00:00" would be sufficient and more accurate to reflect the real format, or perhaps "88:88 - 88:88" if we want to ensure the maximum possible width with proportional fonts.

Maybe this was my mistake, as I tested these changes on an image that already had some differences from the original image.

However, I found a solution for the original Images that maintains the correct "HH:MM - HH:MM" format while ensuring the text is fully displayed. Instead of relying solely on "00:00 - 00:000" to increase the width, I modified the code in the _buildOptionEntryAddTimeDisplay function as follows:

elif config.usage.configselection_showadditionaltimedisplay.value == "7":  # begin - end time
    beginTime = localtime(event.getBeginTime())
    endTime = localtime(event.getBeginTime() + event.getDuration())
    addtimedisplay = "%02d:%02d - %02d:%02d" % (beginTime[3], beginTime[4], endTime[3], endTime[4])
    textTpl = "00:00 - 00:00"
    addtimedisplayWidth = self._calcTextWidth(textTpl, font=self.additionalInfoFont, size=eSize(self.getDesktopWith() // 3, 0))
    addtimedisplayWidth = max(addtimedisplayWidth, 180)
if columnStyle and textTpl:
    addtimedisplayWidth = self._calcTextWidth(textTpl, font=self.additionalInfoFont, size=eSize(self.getDesktopWith() // 3, 0))
    addtimedisplayWidth = max(addtimedisplayWidth, 180)
return addtimedisplay, addtimedisplayWidth

With this modification:

• I used "00:00 - 00:00" as the standard textTpl to match the actual "HH:MM - HH:MM" format.
• I added a minimum width constraint (max(addtimedisplayWidth, 180)) to ensure enough space for the full text, regardless of font size or screen resolution.

I tested this solution, and it works well on the original Image without truncating the text, while keeping the correct format. This way, we achieve the goal of calculating the optimal width without needing extra zeros like "00:000", which might not reflect the expected format.

You might also consider making some adjustments to "buildOptionEntry" to ensure it aligns well with this modification.