Ticket #1700 (closed defect: fixed)

Opened 15 years ago

Last modified 15 years ago

Spaces after color name causes incorrect color

Reported by: sergey-feo Owned by: slavazanko
Priority: minor Milestone: 4.7.0-pre4
Component: mc-skin Version: 4.7.0-pre3
Keywords: Cc:
Blocked By: Blocking:
Branch state: Votes for changeset: committed-master

Description

This skin:

[core]

_default_=lightgray;red

gives correct red background. But this skin:

[core]

_default_=lightgray;red....

where "...." is 4 spaces gives incorrect black background without any error messages.

Attachments

0001-Ticket-1700-fixed-skin-color-parser.patch​ (2.1 KB) - added by andrew_b 15 years ago.
Strip leading and trailing spaces in color names read from skin ini file

Change History

Changed 15 years ago by andrew_b

Strip leading and trailing spaces in color names read from skin ini

file

comment:1 Changed 15 years ago by andrew_b

  • Owner set to slavazanko
  • Status changed from new to assigned

Created 1700_space_in_color_name branch. Parent branch: master.
changeset:17e943401052e9db6ef32070b20c0416ce2cde4d

comment:2 Changed 15 years ago by angel_il

  • Votes for changeset set to angel_il

comment:3 Changed 15 years ago by slavazanko

  • Status changed from assigned to accepted
  • Votes for changeset angel_il deleted
  • severity changed from no branch to on review
  • Milestone changed from 4.7 to 4.7.0-pre4

See 28233a5f470da7968f685f2ac61ec7fac214b21f

This is little optimization fix.

comment:4 Changed 15 years ago by andrew_b

  • Votes for changeset set to andrew_b

comment:5 Changed 15 years ago by iNode

  • Votes for changeset changed from andrew_b to andrew_b iNode
  • severity changed from on review to approved

comment:6 Changed 15 years ago by slavazanko

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b iNode to commited-master
  • Resolution set to fixed
  • severity changed from approved to merged

merge changeset:c492abe28c3d443991f200fa62a1d464a7cf39b2

comment:7 Changed 15 years ago by slavazanko

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.