Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
nyalldawson committed Apr 5, 2023
1 parent 7e0e421 commit dcf7a98
Showing 1 changed file with 5 additions and 31 deletions.
36 changes: 5 additions & 31 deletions python/PyQt/PyQt5/QtGui.py
Expand Up @@ -28,41 +28,15 @@ def __qcolor_repr__(self: QColor):
if not self.isValid():
return '<QColor: invalid>'
elif self.spec() == QColor.Rgb:
return '<QColor: RGB {}, {}, {}, {}>'.format(
self.red(),
self.green(),
self.blue(),
self.alpha()
)
return f'<QColor: RGBA {self.red()}, {self.green()}, {self.blue()}, {self.alpha()}>'
elif self.spec() == QColor.Hsv:
return '<QColor: HSV {}, {}, {}, {}>'.format(
self.hsvHue(),
self.hsvSaturation(),
self.value(),
self.alpha()
)
return f'<QColor: HSVA {self.hsvHue()}, {self.hsvSaturation()}, {self.value()}, {self.alpha()}>'
elif self.spec() == QColor.Cmyk:
return '<QColor: CMYK {}, {}, {}, {}, {}>'.format(
self.cyan(),
self.magenta(),
self.yellow(),
self.black(),
self.alpha()
)
return f'<QColor: CMYKA {self.cyan()}, {self.magenta()}, {self.yellow()}, {self.black()}, {self.alpha()}>'
elif self.spec() == QColor.Hsl:
return '<QColor: HSL {}, {}, {}, {}>'.format(
self.hslHue(),
self.hslSaturation(),
self.lightness(),
self.alpha()
)
return f'<QColor: HSLA {self.hslHue()}, {self.hslSaturation()}, {self.lightness()}, {self.alpha()}>'
elif self.spec() == QColor.ExtendedRgb:
return '<QColor: Extended RGB {}, {}, {}, {}>'.format(
self.redF(),
self.greenF(),
self.blueF(),
self.alphaF()
)
return f'<QColor: Extended RGBA {self.redF()}, {self.greenF()}, {self.blueF()}, {self.alphaF()}>'


# PyQt doesn't provide __repr__ for QColor, but it's highly desirable!
Expand Down

0 comments on commit dcf7a98

Please sign in to comment.