<div dir="ltr">On Tue, Apr 9, 2013 at 5:30 PM, Caolán McNamara <span dir="ltr"><<a href="mailto:caolanm@redhat.com" target="_blank">caolanm@redhat.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Mon, 2013-04-08 at 17:52 +0200, Albert Thuswaldner wrote:<br>
<br>
> I almost completed the conversion of the Delete Contents dialog in<br>
> Calc (see patch #1)<br>
<br>
</div>What's missing from your patch is the .ui file itself, do a git<br>
add /path/to/that/file.ui and git commit --amend</blockquote><div><br></div><div style>Oops, updated the patch accordingly. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">
> What I didn't manage to do was to completely kill the resource id<br>
> RID_SCDLG_DELCONT<br>
><br>
</div><div class="im">> How to handle these things are not covered in your tutorial, at least<br>
> I could not find that...<br>
<br>
</div>Yeah, so the removal of .src and .hrc stuff is a happy-sideeffect of<br>
moving to .ui files, but there are some cases where some .src contents<br>
remain (until e.g. we move over to gettext) and some cases where .hrc<br>
defines have been reused for non-resource files purposes (where the<br>
long-term thing will be to just move them into .hxx files)<br>
<br>
But in this particular case it so happens (common enough occurrence)<br>
that the remaining .hrc define can be removed anyway. See attached.</blockquote><div><br></div><div style>Ok, great. Thanks for the explanation, if I come across similar things in the future I guess I can use this as a reference. For this time assume that you commit your patch with these fixes on top of mine?</div>
<div style> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> I am also scratching my head over the hid key in the help file, not<br>
</div>> sure I got that right ( see patch #2 )<br>
<br>
Looks about right to me. What I would do in addition is to take one of<br>
the two duplicated "hid/.uno:Delete" bookmarks and change one to be the<br>
fallback toplevel dialog help e.g.<br>
"hid/modules/scalc/ui/deletecontents/DeleteContentsDialog</blockquote><div><br></div><div style> Ok, I updated patch #2</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
C.<br>
<br>
</font></span></blockquote></div><br></div><div class="gmail_extra" style>Thanks for your help, much appreciated.</div><div class="gmail_extra" style>/Albert</div><div class="gmail_extra"><br></div></div>