Bug tracker

This ticket has been moved to our new tracker. Open Ticket PHPBB3-6084 now.

incomplete prosilver IE rtl fix (fix completed in vcs)

this problem was dealt with several times, and eventually solved by acydburn. unfortunately the solution was incomplete. fortunately, the fix is easy.

SUMMARY:
in IE only, with RTL languages, in the bottom navbar, text and icons overlap.

DISCUSSION:
the problem was solved some time ago for the top navbar by acydburn by moving the class="icon-xxx" from the <a> tag to the <li> tag.
later, acyd made the same change to the bottom (page_footer) navbar. (see http://phpbb.cvs.sourceforge.net/phpbb/ ... f_format=l )

unfortunately, the last change was not complete: it was done for the U_INDEX link only. in order for the rest of the links to appear properly in IE with RTL languages, the same modification should be done for S_WATCH_FORUM_LINK, U_WATCH_TOPIC, U_BOOKMARK_TOPIC, U_BUMP_TOPIC.

in other words, in overall_footer.php, change:
Code: Select all
                    <!-- IF S_WATCH_FORUM_LINK --><li><a href="{S_WATCH_FORUM_LINK}" <!-- IF S_WATCHING_FORUM -->class="icon-unsubscribe"<!-- ELSE -->class="icon-subscribe"<!-- ENDIF --> title="{S_WATCH_FORUM_TITLE}">{S_WATCH_FORUM_TITLE}</a></li><!-- ENDIF -->
                    <!-- IF 
U_WATCH_TOPIC --><li><a href="{U_WATCH_TOPIC}" title="{L_WATCH_TOPIC}" <!-- IF S_WATCHING_TOPIC -->class="icon-unsubscribe"<!-- ELSE -->class="icon-subscribe"<!-- ENDIF -->>{L_WATCH_TOPIC}</a></li><!-- ENDIF -->
                    <!-- IF 
U_BOOKMARK_TOPIC --><li><a href="{U_BOOKMARK_TOPIC}" title="{L_BOOKMARK_TOPIC}" class="icon-bookmark">{L_BOOKMARK_TOPIC}</a></li><!-- ENDIF -->
                    <!-- IF 
U_BUMP_TOPIC --><li><a href="{U_BUMP_TOPIC}" title="{L_BUMP_TOPIC}" class="icon-bump">{L_BUMP_TOPIC}</a></li><!-- ENDIF -->     


to
Code: Select all
                    <!-- IF S_WATCH_FORUM_LINK --><li <!-- IF S_WATCHING_FORUM -->class="icon-unsubscribe"<!-- ELSE -->class="icon-subscribe"<!-- ENDIF -->><a href="{S_WATCH_FORUM_LINK}" title="{S_WATCH_FORUM_TITLE}">{S_WATCH_FORUM_TITLE}</a></li><!-- ENDIF -->
                    <!-- IF 
U_WATCH_TOPIC --><li <!-- IF S_WATCHING_TOPIC -->class="icon-unsubscribe"<!-- ELSE -->class="icon-subscribe"<!-- ENDIF -->><a href="{U_WATCH_TOPIC}" title="{L_WATCH_TOPIC}">{L_WATCH_TOPIC}</a></li><!-- ENDIF -->
                    <!-- IF 
U_BOOKMARK_TOPIC --><li class="icon-bookmark"><a href="{U_BOOKMARK_TOPIC}" title="{L_BOOKMARK_TOPIC}">{L_BOOKMARK_TOPIC}</a></li><!-- ENDIF -->
                    <!-- IF 
U_BUMP_TOPIC --><li class="icon-bump"><a href="{U_BUMP_TOPIC}" title="{L_BUMP_TOPIC}" >{L_BUMP_TOPIC}</a></li><!-- ENDIF -->     

Comments / History

Edited ticket

Action performed by arod-1 on Oct 3rd 2007, 13:59

Edited ticket

Action performed by arod-1 on Oct 3rd 2007, 14:00

Edited ticket

Action performed by arod-1 on Oct 3rd 2007, 14:25

Edited ticket

Action performed by arod-1 on Oct 3rd 2007, 14:26

Posted by aloooha on Oct 4th 2007, 13:25

What about the subforums list?
The names are written over the icons
sub.GIF (4.96 KB)

Posted by aloooha on Oct 4th 2007, 13:32

This is the code in forumlist_body.html:

****************************************
<!-- IF not forumrow.S_IS_CAT -->
<li class="row">
<dl class="icon" style="background-image: url({forumrow.FORUM_FOLDER_IMG_SRC}); background-repeat: no-repeat;">
<dt>
<!-- IF forumrow.FORUM_IMAGE --><span class="forum-image">{forumrow.FORUM_IMAGE}</span><!-- ENDIF -->
<a href="{forumrow.U_VIEWFORUM}" class="forumtitle">{forumrow.FORUM_NAME}</a><br />
{forumrow.FORUM_DESC}
<!-- IF forumrow.MODERATORS -->
<br /><strong>{forumrow.L_MODERATOR_STR}:</strong> {forumrow.MODERATORS}
<!-- ENDIF -->
<!-- IF forumrow.SUBFORUMS --><br /><strong>{forumrow.L_SUBFORUM_STR}</strong> {forumrow.SUBFORUMS}<!-- ENDIF -->
</dt>
<!-- IF forumrow.CLICKS -->
<dd class="redirect"><span>{L_REDIRECTS}: {forumrow.CLICKS}</span></dd>
<!-- ELSEIF not forumrow.S_IS_LINK -->
<dd class="topics">{forumrow.TOPICS} <dfn>{L_TOPICS}</dfn></dd>
<dd class="posts">{forumrow.POSTS} <dfn>{L_POSTS}</dfn></dd>
<dd class="lastpost"><span>
<!-- IF forumrow.LAST_POST_TIME --><dfn>{L_LAST_POST}</dfn> {L_POST_BY_AUTHOR} {forumrow.LAST_POSTER_FULL}
<a href="{forumrow.U_LAST_POST}">{LAST_POST_IMG}</a> <br />{L_POSTED_ON_DATE} {forumrow.LAST_POST_TIME}<!-- ELSE -->{L_NO_POSTS}<!-- ENDIF --></span>
</dd>
<!-- ENDIF -->
</dl>
</li>
<!-- ENDIF -->
****************************************

This is the part that prints the list:
****************************************
<!-- IF forumrow.SUBFORUMS --><br /><strong>{forumrow.L_SUBFORUM_STR}</strong> {forumrow.SUBFORUMS}<!-- ENDIF -->
****************************************

No <li> or <a> tags there.

What should we change (here or in the CSS files) to correct the display?

Changed ticket status from "New" to "Fix in progress"

Action performed by Acyd Burn (Server Manager) on Oct 4th 2007, 14:38

Assigned ticket to user "Acyd Burn"

Action performed by Acyd Burn (Server Manager) on Oct 4th 2007, 14:38

Posted by Acyd Burn (Server Manager) on Oct 4th 2007, 14:39

Please do not use this report for your subforum list issues... this is about the IE rtl fix previously checked in.

Changed ticket status from "Fix in progress" to "Fix completed in CVS"

Action performed by Acyd Burn (Server Manager) on Oct 4th 2007, 15:05

Linked ticket with changeset: r8135

Action performed by Anonymous (I am too lazy to register) on Oct 4th 2007, 15:08

Posted by aloooha on Oct 4th 2007, 19:04

Sorry - I asked it again and in a new issue - Will I get an answer there?
http://www.phpbb.com/bugs/phpbb3/ticket ... t_id=14631

Ticket details

Related SVN changesets