? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
8 Apr 2021

Summary of Changes

The view that displays a received private message is very hard to read. The background that is applied is so light it just merges into the background. This pr just adds a rounded border.

Adds a border around the message fields.

Testing Instructions

Create two users with admin access
Login as user1 and send a private message to user2
Logout as user1 and log back in as user2 to read the message

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Documentation Changes Required

none

avatar brianteeman brianteeman - open - 8 Apr 2021
avatar brianteeman brianteeman - change - 8 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Apr 2021
Category Administration com_messages
avatar ceford ceford - test_item - 9 Apr 2021 - Tested successfully
avatar ceford
ceford - comment - 9 Apr 2021

I have tested this item successfully on 9b08162

As this is a visibility issue I was interested in what Axe said - just one issue on the message page:

Ensures all page content is contained by landmarks

Element location

<div class="sidebar-toggle item item-level-1"> <a id="menu-collapse" href="#" aria-label="Toggle Menu"> <span id="menu-collapse-icon" class="icon-fw icon-toggle-on" aria-hidden="true"></span> <span class="sidebar-item-title">Toggle Menu</span> </a> </div>

Just something to ponder!


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33074.
avatar sandramay0905 sandramay0905 - test_item - 9 Apr 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 9 Apr 2021

I have tested this item successfully on 9b08162


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33074.

avatar brianteeman
brianteeman - comment - 9 Apr 2021

Thanks for testing.

@ceford that is a separate issue. PLEASE dont pollute issues with unrelated things just create a separate issue

avatar Quy Quy - change - 9 Apr 2021
Status Pending Ready to Commit
Labels Added: ?
avatar Quy
Quy - comment - 9 Apr 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33074.

avatar Quy Quy - close - 9 Apr 2021
avatar Quy Quy - merge - 9 Apr 2021
avatar Quy Quy - change - 9 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-09 15:09:41
Closed_By Quy
Labels Added: ?
avatar Quy
Quy - comment - 9 Apr 2021

Thanks

avatar brianteeman
brianteeman - comment - 9 Apr 2021

thanks for testing and merging

Add a Comment

Login with GitHub to post a comment