Project

General

Profile

Bug #3424

Service Mapping Improvements ("Map All")

Added by Prof Yaffle over 5 years ago. Updated over 5 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
Category:
Service Mapping
Target version:
Start date:
2015-12-17
Due date:
% Done:

100%

Estimated time:
Found in version:
4.1-1228~g07f77f6
Affected Versions:

Description

Part bug, part FR, I think...

Recent builds removed the "Map All" function, instead leaving the user to do a Ctrl+A to select all services before clicking the "Map" button. If you don't do this, you're presented with a LovCombo (list-of-values combo) box in which you'd need to manually select each item to map.

Two suggestions:

1. As of the build below, the button (Configuration -> DVB Inputs -> Services) is still labelled "Map All", which it no longer does. On Configuration -> Channel/EPG -> Channels, it's labelled more correctly as "Map Services". This is easy to fix... however...

2. From what I read, the LovCombo supports a selectAll() and deselectAll() method. These would be useful buttons in the mapping dialog as it allows the user to easily manipulate the service list without cancelling, re-selecting, and re-entering the service mapping window. Alternatively, there could be separate "Map selected" and "Map all" buttons on the main UI, although I personally feel that this is shifting function to the wrong panel.

I'd try to add it myself, but I can't understand how the UI elements are built from the C code any more :(

Associated revisions

Revision f65b842d (diff)
Added by Jaroslav Kysela over 5 years ago

WEBUI: Fix 'map all' functionality for channels/services grids, fixes #3424

History

#1

Updated by Prof Yaffle over 5 years ago

There was some discussion on IRC as to whether leaving all services unselected gave you a "map all" function.

That seemed like an odd UI choice, but - on testing - it does not seem to be the case anyway. Selecting "Map Services" or "Map All" with nothing selected and no services are mapped, as would be expected. You need to do the Ctrl+A shuffle beforehand.

#2

Updated by Jaroslav Kysela over 5 years ago

  • Target version set to 4.2
#3

Updated by Jaroslav Kysela over 5 years ago

  • Status changed from New to Fixed
  • % Done changed from 0 to 100
#4

Updated by Jaroslav Kysela over 5 years ago

  • Status changed from Fixed to New
  • % Done changed from 100 to 0

Fixed in v4.1-1327-gf65b842 (created two menu items - 'selected' and 'all').

#5

Updated by Jaroslav Kysela over 5 years ago

  • % Done changed from 0 to 100
#6

Updated by Jaroslav Kysela over 5 years ago

  • Status changed from New to Fixed

Also available in: Atom PDF