[digikam] [Bug 320462] New: Button for resetting curve in RAW import tool is very small

[Dmitry]

https://bugs.kde.org/show_bug.cgi?id=320462

            Bug ID: 320462
           Summary: Button for resetting curve in RAW import tool is very
                    small
    Classification: Unclassified
           Product: digikam
           Version: 3.3.0
          Platform: Other
                OS: Linux
            Status: UNCONFIRMED
          Severity: wishlist
          Priority: NOR
         Component: Image Editor
          Assignee: [hidden email]
          Reporter: [hidden email]

Button for resetting curve to linear is too small, so, it's difficalt to find
it (if user don't know where the button is located). This patch makes button
bigger. Please, see screenshots.

Reproducible: Always

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Dmitry]

https://bugs.kde.org/show_bug.cgi?id=320462

--- Comment #1 from Dmitry <[hidden email]> ---
Created attachment 80175
  --> https://bugs.kde.org/attachment.cgi?id=80175&action=edit
Small button

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Dmitry]

https://bugs.kde.org/show_bug.cgi?id=320462

--- Comment #2 from Dmitry <[hidden email]> ---
Created attachment 80176
  --> https://bugs.kde.org/attachment.cgi?id=80176&action=edit
New large button

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Dmitry]

https://bugs.kde.org/show_bug.cgi?id=320462

--- Comment #3 from Dmitry <[hidden email]> ---
Review request: https://git.reviewboard.kde.org/r/110559/

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=320462

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]

--- Comment #4 from Gilles Caulier <[hidden email]> ---
With your patch applied, go to Image Curves tool. What's the result in settings
view ?

Why not to put a contextual menu over curve ?

Gilles Caulier

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=320462

--- Comment #5 from Gilles Caulier <[hidden email]> ---
Ah no, sorry, your patch touch only RawSettingsBox, not main Curve Widget...

So it's fine for me...

Gilles Caulier

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=320462

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/digi
                   |                            |kam/8917940468074dcd29d6067
                   |                            |fe905b011b292dd63

--- Comment #6 from Gilles Caulier <[hidden email]> ---
Git commit 8917940468074dcd29d6067fe905b011b292dd63 by Gilles Caulier.
Committed on 30/05/2013 at 09:44.
Pushed by cgilles into branch 'master'.

Apply patch from Dmitry Ashkadov to inmprove visibility of Cureve reset button
from Raw Import Tool.
REVIEW: 110559

M  +12   -15   utilities/imageeditor/rawimport/rawsettingsbox.cpp

http://commits.kde.org/digikam/8917940468074dcd29d6067fe905b011b292dd63

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=320462

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|                            |3.3.0

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel