[MDL-30898] Removal of .blink CSS selector rule - Moodle Tracker

XML

Word

Printable

Details

Type: Sub-task
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 2.1.1, 2.4
Fix Version/s: 2.4
Component/s: Accessibility, Themes
Labels:
- triaged

Testing Instructions:
Hide

Using Firefox on moodle 2.3, create a text page that contains <div class="blink">This text is blinking</div>

Check the text is blinking (Webkit doesn't seem to support the blinking text decoration)

Upgrade moodle to master and check the text is no longer blinking
Show
Using Firefox on moodle 2.3, create a text page that contains <div class="blink">This text is blinking</div> Check the text is blinking (Webkit doesn't seem to support the blinking text decoration) Upgrade moodle to master and check the text is no longer blinking
Affected Branches:
MOODLE_21_STABLE, MOODLE_24_STABLE
Fixed Branches:
MOODLE_24_STABLE
Pull from Repository:
git://github.com/phalacee/moodle.git
Pull Master Branch:
wip-~~MDL-30898~~-master
Pull Master Diff URL:
https://github.com/phalacee/moodle/compare/master...wip-MDL-30898-master

Description

There are CSS rules defined for .blink (blinking text) and .justifytext (fully justified text). It is not apparent that either of these rules are being used in the standard theme, but they should not be used in most cases as they both have accessibility problems.

Attachments

Activity

People

Assignee:: Jason Fowler

Reporter:: Greg Kraus

Peer reviewer:: Rossiani Wijaya

Integrator:: Aparup Banerjee

Tester:: Adrian Greeve

Participants:: Adrian Greeve, Aparup Banerjee, Eloy Lafuente (stronk7), Greg Kraus, Jason Fowler, Mary Evans, Rossiani Wijaya

Component watchers:: Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Bas Brands

Votes:: 0 Vote for this issue

Watchers:: 2 Start watching this issue

Dates

Created:: 23/Dec/11 12:47 PM

Updated:: 30/Sep/13 9:07 PM

Resolved:: 15/Sep/12 10:38 AM

Fix Release Date:: 3/Dec/12