Page MenuHomePhabricator

hex color defs handling
Closed, DuplicatePublic

Description

Author: gonia

Description:
a patch to allow xrgb color specification in addition to rgb
xrgb allows to put 6-digit hexadecimal string as a color value instead
of rgb's decimal fraction notation, (side effect: rgb and hsv
handling is merged, because they only differ in name)


Version: unspecified
Severity: enhancement

Details

Reference
bz4168

Event Timeline

bzimport raised the priority of this task from to Low.Nov 21 2014, 8:57 PM
bzimport added a project: EasyTimeline.
bzimport set Reference to bz4168.
bzimport added a subscriber: Unknown Object (MLST).

gonia wrote:

patch to get xrgb (hex) notation active for color definitions

Attached:

Patch review needs to wait on a major EasyTimeline.pl update from Erik;
don't know if affected code will change.

avarab wrote:

(In reply to comment #1)

Created an attachment (id=1133) [edit]
patch to get xrgb (hex) notation active for color definitions

You don't need "((?:rgb|hsb))", you can just use "(rgb|hsb)"

Assigned to extension developer.

Perhaps with bug 27156 it will get done.

sumanah wrote:

CC'ing Tim Landscheidt to see if he'd like to address this in his revamp.

sumanah wrote:

Piotr, I'm sorry for the wait in response.

If the issue in this bug report is still a problem, you might wish to revise your patch and get it reviewed faster by using Developer access https://www.mediawiki.org/wiki/Developer_access to submit it directly into our Git source control system. Thanks!

Tim, do you think that you will be addressing this in your revamp of this extension?