Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#35 closed bug (fixed)

Background color picker gadget in Wookie chat produces Gadget/Image selector

Reported by: marcus.gerards@… Owned by: Thore Böckelmann
Priority: normal Milestone: 3.9-2014R2
Component: Backgroundadjust.mui Version: 3.9-2014R1
Severity: minor Keywords:
Cc: OS Platform: AmigaOS3
Blocked By: Blocking:
Release Notes:

Description

Summary

Background color picker gadget in Wookie chat produces Gadget/Image selector. All settings of the first row are affected, the rest will call up the color picker (as expected)

Steps to reproduce

  1. Start WookieCHat (OS3.9)
  2. Select settings→color settings
  3. Choose background-color settings

Expected results

Color picker should show up

Actual results

Image selector shows up.

Regression

Notes

This did not happen with MUI3.8, so I blame MUI ;)

Change History (5)

comment:1 Changed 5 years ago by Thore Böckelmann

Component: undefinedBackgroundadjust.mui
Milestone: MUI 3.9-2014R2
Priority: undecidednormal

comment:2 Changed 5 years ago by Thore Böckelmann

Owner: set to Thore Böckelmann
Resolution: fixed
Status: newclosed

In 3563:

  • Backgroundadjust.c: added a workaround for WookieChat which uses a private attribute to initialize the pen adjustment which in turn caused the class to show the image selection instead of the pen adjustment. This closes #35.

comment:3 Changed 5 years ago by Thore Böckelmann

In 3566:

  • Backgroundadjust.c: refined the wrong background selection of WookieChat a bit more to work correctly. This refs #35.

comment:4 Changed 5 years ago by Thore Böckelmann

Let me finally point out that this was a bug in MUI and WookieChat. WC is using the private class Popimage.mui and fails to provide some required attributes. On the other hand MUI's pen/image adjustment classes have changed fundamentally and there is only one single left which acts as a wrapper for all the old classes. And this wrapper got confused by WC's missing attribute.

comment:5 Changed 4 years ago by Thore Böckelmann

Milestone: MUI 3.9-2014R23.9-2014R2

Milestone renamed

Note: See TracTickets for help on using tickets.