[3.0] Theme #7933
[3.0] Theme #7933
Conversation
Themes/default/index.template.php
Outdated
| <li> | ||
| <a href="', Config::$scripturl, '?action=profile"', !empty(Utils::$context['self_profile']) ? ' class="active"' : '', ' id="profile_menu_top">'; | ||
|
|
||
| if (!empty(User::$me->avatar)) | ||
| echo User::$me->avatar['image']; | ||
|
|
||
| echo '<span class="textmenu">', User::$me->name, '</span></a> | ||
| echo '</a> |
There was a problem hiding this comment.
If we do this for the avatar image, shouldn't we use a alt text for the name?
There was a problem hiding this comment.
Should use href or url then.
Or should 'image' provide it by default? It would be good in case an external avatar is missing in other parts of the forum.
There was a problem hiding this comment.
I think either is valid. I believe it just needs something to be valid for screen readers. Since its a link to the users profile and we removed the name, the alt should have it.
There was a problem hiding this comment.
Then I think 'image' should provide the alt text by default. An external avatar could suddenly not load or disappear, would be good to have it.
live627
force-pushed
the
theme
branch
4 times, most recently
from
b7c6385
to
3a78afe
Compare
|
I notice that a few forms would disable textareas on submit, such as the posting form. I've changed this to do all forms by using the new HTML property to make the entire form inert. |
live627
force-pushed
the
theme
branch
2 times, most recently
from
3ba4ff5
to
c823a49
Compare
|
Icons in the admin area are not indented enough: Unable to load the '' template at Forum Maintenance - Routine: SCEditor doesn't have a dark theme? Instead of "X Posts" and "Y Topics", maybe it would still be better to keep "Posts: X" and "Topics: Y" as it is now? |
I’m working on this
Can be fixed later too, but also requires some tweaks.
I don’t have a preference, we might discuss it later for sure. |
live627
force-pushed
the
theme
branch
2 times, most recently
from
2322f68
to
ada494b
Compare
|
@live627 Can you rebase this or merge and fix conflicts? |
|
We have this pending Will start sending more in upcoming weeks, but it could be merged, rest of changes are more focused so can be in separated PR's |
|
Once I can get around to testing and merging that, I'll rebase this branch again |
Themes/default/Reminder.template.php
Outdated
| <div class="windowbg form_grid"> | ||
| <p class="descbox">', Lang::$txt['authentication_options'], ':</p> | ||
| <p> | ||
| <input type="radio" name="reminder_type" id="reminder_type_email" value="email" checkeiv></label> |
Themes/default/Register.template.php
Outdated
| <input type="submit" value="', Lang::$txt['save'], '" name="save_reserved_names" tabindex="', Utils::$context['tabindex']++, '" class="button"> | ||
| <div class="form_grid"> | ||
| <div> | ||
| <label for="matchword">', Lang::$txt['admin_match_whole'], '</div> |
There was a problem hiding this comment.
<label>...</div> => <label>...</label>?
|
@jdarwood007 I moved some of the login javascript to its own file but don't know how to test cors requests. |
|
When this is closer to ready I can check it. You need to run 2 domains (or 2 subdomains) and have the forum on one and the script on another. A simple SSI page where you can open the user area popup is good enough for most things. Sending credentials such as on the login form may also need to be tested, but when you can open the user area, the CORS request works. |
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: John Rayes <[email protected]>
|
I opened the pr on your branch |
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
Signed-off-by: Diego Andrés <[email protected]>
This is the new theme started by @TwitchisMental
I still need to convert several form grids to use CSS grids instead of floats. Also the grids abuse definition list markup, something tat has always annoyed me from the very beginning.
Submitting as a draft now to get early feedback.