Skip to content

Commit

Permalink
Remove redundant "All Programs - " text in Watch Recordings screen.
Browse files Browse the repository at this point in the history
Fixes #11027.  Removes the "DispRecGroupAsAllProg" setting (Show
filter name instead of "All Programs"), and always uses "All Programs"
as the first group name.  Removes the unnecessary prefix from the
titles of the group entries.

Fixes the (not-yet documented) "group" widget of the watchrecordings
window in recordings-ui.xml to always hold the current group filter
name.  Themers are suggested to display the current group filter in a
textarea, e.g.:

  <value>Watch Recordings</value>
  <template>Watch Recordings - %1</template>
  • Loading branch information
stichnot committed Dec 11, 2012
1 parent e5dd785 commit ba2c783
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 36 deletions.
12 changes: 0 additions & 12 deletions mythtv/programs/mythfrontend/globalsettings.cpp
Expand Up @@ -130,17 +130,6 @@ static HostCheckBox *RememberRecGroup()
return gc;
}

static HostCheckBox *UseGroupNameAsAllPrograms()
{
HostCheckBox *gc = new HostCheckBox("DispRecGroupAsAllProg");
gc->setLabel(QObject::tr("Show filter name instead of \"All Programs\""));
gc->setValue(false);
gc->setHelpText(QObject::tr("If enabled, use the name of the display "
"filter currently applied in place of the term \"All "
"Programs\" in the playback screen."));
return gc;
}

static HostCheckBox *PBBStartInTitle()
{
HostCheckBox *gc = new HostCheckBox("PlaybackBoxStartInTitle");
Expand Down Expand Up @@ -3285,7 +3274,6 @@ PlaybackSettings::PlaybackSettings()
pbox2->addChild(DisplayRecGroup());
pbox2->addChild(QueryInitialFilter());
pbox2->addChild(RememberRecGroup());
pbox2->addChild(UseGroupNameAsAllPrograms());
addChild(pbox2);

VerticalConfigurationGroup* pbox3 = new VerticalConfigurationGroup(false);
Expand Down
31 changes: 9 additions & 22 deletions mythtv/programs/mythfrontend/playbackbox.cpp
Expand Up @@ -385,7 +385,6 @@ PlaybackBox::PlaybackBox(MythScreenStack *parent, QString name, BoxType ltype,
m_type(ltype),
m_watchListAutoExpire(false),
m_watchListMaxAge(60), m_watchListBlackOut(2),
m_groupnameAsAllProg(false),
m_listOrder(1),
// Recording Group settings
m_groupDisplayName(ProgramInfo::i18n("All Programs")),
Expand Down Expand Up @@ -426,7 +425,6 @@ PlaybackBox::PlaybackBox(MythScreenStack *parent, QString name, BoxType ltype,
m_watchListAutoExpire= gCoreContext->GetNumSetting("PlaybackWLAutoExpire", 0);
m_watchListMaxAge = gCoreContext->GetNumSetting("PlaybackWLMaxAge", 60);
m_watchListBlackOut = gCoreContext->GetNumSetting("PlaybackWLBlackOut", 2);
m_groupnameAsAllProg = gCoreContext->GetNumSetting("DispRecGroupAsAllProg", 0);

bool displayCat = gCoreContext->GetNumSetting("DisplayRecGroupIsCategory", 0);

Expand Down Expand Up @@ -472,8 +470,7 @@ PlaybackBox::PlaybackBox(MythScreenStack *parent, QString name, BoxType ltype,
m_recGroupIdx = -1;
m_recGroupType.clear();
m_recGroupType[m_recGroup] = (displayCat) ? "category" : "recgroup";
if (m_groupnameAsAllProg)
m_groupDisplayName = ProgramInfo::i18n(m_recGroup);
m_groupDisplayName = ProgramInfo::i18n(m_recGroup);

fillRecGroupPasswordCache();

Expand Down Expand Up @@ -639,21 +636,11 @@ void PlaybackBox::updateGroupInfo(const QString &groupname,
InfoMap infoMap;
int countInGroup;

if (groupname.isEmpty())
{
countInGroup = m_progLists[""].size();
infoMap["title"] = m_groupDisplayName;
infoMap["group"] = m_groupDisplayName;
infoMap["show"] = ProgramInfo::i18n("All Programs");
}
else
{
countInGroup = m_progLists[groupname].size();
infoMap["title"] = QString("%1 - %2").arg(m_groupDisplayName)
.arg(grouplabel);
infoMap["group"] = m_groupDisplayName;
infoMap["show"] = grouplabel;
}
infoMap["group"] = m_groupDisplayName;
infoMap["title"] = grouplabel;
infoMap["show"] =
groupname.isEmpty() ? ProgramInfo::i18n("All Programs") : grouplabel;
countInGroup = m_progLists[groupname].size();

if (m_artImage[kArtworkFanart])
{
Expand Down Expand Up @@ -841,6 +828,7 @@ void PlaybackBox::UpdateUIListItem(MythUIButtonListItem *item,
InfoMap infoMap;

pginfo->ToMap(infoMap);
infoMap["group"] = m_groupDisplayName;
ResetMap(m_currentMap);
SetTextFromMap(infoMap);
m_currentMap = infoMap;
Expand Down Expand Up @@ -1349,7 +1337,7 @@ void PlaybackBox::UpdateUIGroupList(const QStringList &groupPreferences)

QString displayName = groupname;
if (displayName.isEmpty())
displayName = m_groupDisplayName;
displayName = ProgramInfo::i18n("All Programs");

item->SetText(displayName, "name");
item->SetText(displayName);
Expand Down Expand Up @@ -4493,8 +4481,7 @@ void PlaybackBox::setGroupFilter(const QString &recGroup)

m_recGroup = newRecGroup;

if (m_groupnameAsAllProg)
m_groupDisplayName = ProgramInfo::i18n(m_recGroup);
m_groupDisplayName = ProgramInfo::i18n(m_recGroup);

// Since the group filter is changing, the current position in the lists
// is meaningless -- so reset the lists so the position won't be saved.
Expand Down
2 changes: 0 additions & 2 deletions mythtv/programs/mythfrontend/playbackbox.h
Expand Up @@ -371,8 +371,6 @@ class PlaybackBox : public ScheduleCommon
int m_watchListMaxAge;
/// adjust exclusion of a title from the Watch List after a delete
int m_watchListBlackOut;
/// contains "DispRecGroupAsAllProg" setting
bool m_groupnameAsAllProg;
/// allOrder controls the ordering of the "All Programs" list
int m_allOrder;
/// listOrder controls the ordering of the recordings in the list
Expand Down

0 comments on commit ba2c783

Please sign in to comment.