Bug 220563 - Update UI for PaletteTemplateEntries
Summary: Update UI for PaletteTemplateEntries
Status: RESOLVED FIXED
Alias: None
Product: GEF
Classification: Tools
Component: GEF-Legacy GEF (MVC) (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4.0 (Ganymede) M6   Edit
Assignee: Cherie Revells CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-02-27 10:05 EST by Cherie Revells CLA
Modified: 2008-09-18 13:36 EDT (History)
3 users (show)

See Also:


Attachments
patch to update UI for palette templates (10.36 KB, patch)
2008-03-10 13:58 EDT, Cherie Revells CLA
ahunter.eclipse: iplog+
Details | Diff
Patch that will allow you to test in a GEF logic diagram (3.24 KB, patch)
2008-03-10 13:59 EDT, Cherie Revells CLA
ahunter.eclipse: iplog+
Details | Diff
partial screenshot (34.30 KB, image/png)
2008-03-26 18:23 EDT, Nitin Dahyabhai CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Cherie Revells CLA 2008-02-27 10:05:18 EST
The selection and hover UI needs to also be updated for PaletteTemplateEntries.  I missed this when solving Bug 133385.
Comment 1 Cherie Revells CLA 2008-03-10 13:58:28 EDT
Created attachment 92062 [details]
patch to update UI for palette templates

I added a patch that modifies the way palette templates behave.  The templates now get highlighted in blue like palette tools when selected.  I did not change the color to orange on a template when hovering over as there was no hover feedback previously either.  I also fixed the indenting of palette templates to match that of palette tools.

I cleaned up the code in DetailedLabelFigure as the focus rectangle and blue selection rectangle need not be drawn anymore.
Comment 2 Cherie Revells CLA 2008-03-10 13:59:25 EDT
Created attachment 92063 [details]
Patch that will allow you to test in a GEF logic diagram
Comment 3 Cherie Revells CLA 2008-03-10 14:00:31 EDT
Anthony, this one is ready for review.
Comment 4 Anthony Hunter CLA 2008-03-10 17:18:27 EDT
Committed to HEAD.

The Snippits view should be verified with the new palette in this week's GEF integration build.
Comment 5 Nitin Dahyabhai CLA 2008-03-26 18:23:48 EDT
Created attachment 93711 [details]
partial screenshot

(In reply to comment #4)
> Committed to HEAD.
> 
> The Snippits view should be verified with the new palette in this week's GEF
> integration build.

Yes, but while it used to adhere to the system colors, now it just doesn't.  The blue used is seen nowhere else in my color palette or workbench.