Opened 16 years ago
Closed 15 years ago
#5272 closed patch (fixed)
MythWeb Advanced Scheduling Options
Reported by: | Owned by: | Rob Smith | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | mythweb | Version: | 0.21 |
Severity: | medium | Keywords: | easy |
Cc: | Ticket locked: | no |
Description
There are some problems with Advanced Scheduling Options in MythWeb, not really serious but annoying: I made following changes to file modules/tv/tmpl/default/_advanced
(<?php echo '<a href="#" onclick="toggle_advanced(false)" id="hide_advanced"'; if (!$_SESSION['tv']['show_advanced_schedule']) echo ' style="display: none"'; echo '>', t('Hide'), '</a>', '<a href="#" onclick="toggle_advanced(true)" id="show_advanced"'; if ($_SESSION['tv']['show_advanced_schedule']) echo ' style="display: none"'; echo '>', t('Display'), '</a>'; ?>)
1) I added href="#" so that correct mouse cursor appears 2) I Changed Label from Show to Display to avoid translation issues (by example in German Show is translated by Sendung, what is not correct in this context)
Finally I added a Capital Letter fo german traslantion of Display, to be consistent to other similar labels like Hide.
Attachments (1)
Change History (5)
comment:1 Changed 16 years ago by
Owner: | changed from xris to Rob Smith |
---|---|
Status: | new → accepted |
comment:2 Changed 16 years ago by
Status: | accepted → infoneeded |
---|
Changed 16 years ago by
Attachment: | ticket_5572.diff added |
---|
comment:3 Changed 16 years ago by
Keywords: | easy added |
---|---|
Status: | infoneeded → assigned |
Type: | defect → patch |
Patch against trunk is attached. Untested.
The translation change is already in trunk it seems.
Care to make this as a patch?
svn diff > foo.patch
and then attaching it to the ticket?