<div dir="ltr">Your change is in trunk.<div><br></div><div>Thank you for the contribution.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 25, 2018 at 5:33 PM, John-Reed Maffeo <span dir="ltr"><<a href="mailto:jrmaffeo@gmail.com" target="_blank">jrmaffeo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I have no opinion, I just copied what was used in #<span style="color:rgb(0,0,0)">ToolIcons. #HelpIcons currently uses #Dictionary. It makes sense that they would all used the same pattern. I have not made any contributions in quite a while and I am not up to speed with the community development model. <span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">I checked the wiki for info about the community development model, but I am not quite sure how to use it. I can access the Repository at <a href="http://source.squeak.org/inbox" target="_blank">http://source.squeak.org/inbox</a> in Monticello Browser, but I am not sure what to do next (or even if I should).</span></span><div><font color="#000000"><br></font></div><div><font color="#000000">The attached change set updates the class method #icon in #ToolIcons and #MenuIcons to be consistent with the use of #Dictionary in #HelpIcons. All three should now be the same.</font></div><div><font color="#000000"><br></font></div><div><font color="#000000">My only test was to #do </font></div><div><font color="#000000"><br></font></div><div><font color="#000000"><div>ToolIcons icons explore.</div><div>MenuIcons icons explore.</div><div>HelpIcons icons explore. </div><div><br></div><div>in a Workspace. Results appeared to be correct.</div></font></div><div><font color="#000000"><br></font><div><font color="#000000"><br></font></div><div><font color="#000000">Cheers,<br></font><pre style="color:rgb(0,0,0);font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;word-spacing:0px;text-decoration-style:initial;text-decoration-color:initial"></pre><div><span style="color:rgb(0,0,0)">jrm</span></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 25, 2018 at 1:58 PM, Chris Muller <span dir="ltr"><<a href="mailto:asqueaker@gmail.com" target="_blank">asqueaker@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think it's wrong for it to be an IdentityDictionary, even though the<br>
keys are Symbols.  We should convert it to a plain Dictionary.<br>
<div class="m_7687172575878887016HOEnZb"><div class="m_7687172575878887016h5"><br>
On Tue, Apr 24, 2018 at 7:56 PM, John-Reed Maffeo <<a href="mailto:jrmaffeo@gmail.com" target="_blank">jrmaffeo@gmail.com</a>> wrote:<br>
> The class method #icons is missing from #MenuIcons.<br>
><br>
> I have attached my tested proposal for a replacement.<br>
><br>
> - jrm<br>
><br>
><br>
><br>
<br>
</div></div></blockquote></div><br></div>
</blockquote></div><br></div>