-
-
Notifications
You must be signed in to change notification settings - Fork 408
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Working on #2972 Template Conversion to Twig Format (memberlist.php) #2998
Working on #2972 Template Conversion to Twig Format (memberlist.php) #2998
Conversation
Part 1 - Member list
<tr> | ||
<td class="thead" colspan="{{ memberlist.colspan }}"> | ||
<div class="float_right"> | ||
<a href="memberlist.php?username_match=begins&username={{ lang.a }}" style="padding: 5px;">{{ lang.a }}</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might make sense to use a loop here if possible.
<tr> | ||
<td class="tcat" width="1%"><span class="smalltext"><strong>{{ lang.avatar }}</strong></span></td> | ||
<td class="tcat"> | ||
<span class="smalltext"><a href="memberlist.php?perpage={{ mybb.input.perpage }}{{ memberlist.search_url }}&sort=username&order=ascending"><strong>{{ lang.username }}</strong></a> {% if memberlist.orderarrow.username %}<span class="smalltext">[<a href="memberlist.php?perpage={{ mybb.input.perpage }}{{ memberlist.search_url }}&sort=username&order={{ memberlist.oppsortnext }}">{{ memberlist.oppsort }}</a>]</span>{% endif %}</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we split these long lines into multiple lines?
@PaulBender Can you update #2972 with all the templates you've changed and got merged please? |
Working on #2972
This PR is for the member list (memberlist.php).