PeterPanettone 158 Posted March 15, 2019 (edited) I have filed a new feature suggestion for MMX: The "Find Identifier" popup window should have an "implementation" marking: This would be very useful as it would allow seeing at a glance which identifiers are above or inside the implementation section! Edited March 15, 2019 by PeterPanettone Share this post Link to post
Dinar 22 Posted March 15, 2019 (edited) I think Parnassus Bookmarks editor window like would be more nice implementation. Just instead of bookmarks, there will be displayed the founded identifier You may read more about it at this article: https://community.idera.com/developer-tools/b/blog/posts/new-productivity-tooling-in-rad-studio-10-3-1-bookmarks Edited March 15, 2019 by Dinar Share this post Link to post
FredS 138 Posted March 15, 2019 6 hours ago, PeterPanettone said: inside the implementation Well, or we could just continue to use the preceding class name as the indicator... procedure DoStuff; // interface procedure TMyStuff.DoStuff; // implementation Share this post Link to post
PeterPanettone 158 Posted March 15, 2019 (edited) 3 minutes ago, FredS said: Well, or we could just continue to use the preceding class name as the indicator... procedure DoStuff; // interface procedure TMyStuff.DoStuff; // implementation The "implementation" marking is much better visible and therefore more INTUITIVE. Edited March 15, 2019 by PeterPanettone Share this post Link to post
PeterPanettone 158 Posted March 15, 2019 (edited) 26 minutes ago, Dinar said: I think Parnassus Bookmarks editor window like would be more nice implementation. Just instead of bookmarks, there will be displayed the founded identifier 1 Writing "implementation section" in each and every found item is useless and takes too much space on the list. One single implementation marking like in my example is enough to clearly see which found items are above the implementation keyword and which ones are below the implementation keyword, as there is maximally only ONE implementation keyword in a unit. Edited March 15, 2019 by PeterPanettone Share this post Link to post
Uwe Raabe 2063 Posted March 15, 2019 8 hours ago, PeterPanettone said: The "Find Identifier" popup window should have an "implementation" marking: How is this supposed to work when the number of visible lines is less than the number of lines in the interface/implementation section? A simple marker line (I would suggest something like a plain horizontal line to avoid wasting a full text line) will scroll out of view. I was thinking of different text and/or background colors for interface and implementation, but the styling system doesn't really like custom coloring. It often looks pretty poor in at least one of the standard themes. Perhaps we can adopt some of the syntax highlighter settings? This would honor the fact that the editor can have a different color scheme than the IDE itself. Share this post Link to post
PeterPanettone 158 Posted March 15, 2019 (edited) 1 hour ago, Uwe Raabe said: How is this supposed to work when the number of visible lines is less than the number of lines in the interface/implementation section? A simple marker line (I would suggest something like a plain horizontal line to avoid wasting a full text line) will scroll out of view. I was thinking of different text and/or background colors for interface and implementation, but the styling system doesn't really like custom coloring. It often looks pretty poor in at least one of the standard themes. Perhaps we can adopt some of the syntax highlighter settings? This would honor the fact that the editor can have a different color scheme than the IDE itself. 1 Good point. Yes, different text and/or background colors for interface and implementation would be nice. The aim is to somehow indicate the scope of each found identifier. But then it would be logical to differentiate also between other scopes? What do you think? Edited March 15, 2019 by PeterPanettone Share this post Link to post
Uwe Raabe 2063 Posted March 16, 2019 What about showing symbols as used in the contents tree? They can be placed somewhere between the Match, Line or Source column. The Yellow Plus symbol signals interface while the Blue Minus stands for implementation. That would waste only a little bit of real estate. Not sure what you refer to with "other scopes". Can you name some? 1 Share this post Link to post
dummzeuch 1515 Posted March 16, 2019 (edited) I think the symbols are a good idea. But why the plus an minus? In the contents tree they signal that you can expand the node (I guess, I haven't installed you tool on this computer). Since the colours are quite easy to distinguish, use the circles without the plus / minus characters. Edited March 16, 2019 by dummzeuch Share this post Link to post
Uwe Raabe 2063 Posted March 16, 2019 The Plus and Minus signs are taken from the UML specification for public and private. The colors are just a visual enhancement. These icons are used all over the place, especially for the Delphi class visibility: They are not related to expanding or collapsing the tree structure (spot the down arrow in front of the Module node in the content tree screenshot above). 1 Share this post Link to post
Jacek Laskowski 57 Posted March 16, 2019 Maybe green and gray circle would be enough? Without plus or minus sign? Green for interface, gray for implementation (gray as in shadow, hidden). Share this post Link to post
Uwe Raabe 2063 Posted March 16, 2019 18 minutes ago, Jacek Laskowski said: Maybe green and gray circle would be enough? Without plus or minus sign? As I said: those two icons are already used elsewhere to mark the interface and implementation section and should thus be immediately recognized. Why should I introduce alternative icons here? 2 Share this post Link to post
PeterPanettone 158 Posted March 16, 2019 24 minutes ago, Uwe Raabe said: As I said: those two icons are already used elsewhere to mark the interface and implementation section and should thus be immediately recognized. Why should I introduce alternative icons here? I like this idea. Share this post Link to post