On 06/08/2012 12:24 AM, Joel Madero wrote:
Thanks for pointing that out, correction:
https://bugs.freedesktop.org/show_bug.cgi?id=43192
On Thu, Jun 7, 2012 at 10:20 PM, Daniel Bankston
<daniel.dev.libreoff...@gmail.com
<mailto:daniel.dev.libreoff...@gmail.com>> wrote:
On 06/07/2012 10:58 PM, Joel Madero wrote:
Hi All,
Before I push this (which I will do tomorrow). I want to get
preference opinions if possible. Here is the bug report:
https://bugs.freedesktop.org/show_bug.cgi?id=43518
I tend to agree with the complaint (although it's incredibly
minor). Technically "between" should not include the minimum
and maximum numbers put in. Since they are (and I think that
this is preferred) the text should be changed. I opted for
"valid range" and "invalid range". There are other options
including:
between (inclusive) & not between (inclusive) or simply
include range & exclude range
This is really minor but I'm trying to knock off some of these
little bugs that just take a small amount of time to fix. If
you're opposed to any change in language &/or prefer different
wording please respond, otherwise I'm pushing my updates as
soon as possible.
Lastly, on a technical note. What's the easiest way for me to
replace all occurrence of a variable with another name. In
order to be consistent I'll have to update the variable name
along with the actual text name.
Thanks all,
Joel
Hi, Joel,
Maybe I misunderstand, but I think you accidentally linked the
wrong bug report.
Respectfully,
Daniel Bankston
Hi, Joel,
Ironically, I have been working today on ODS's import of Data Validity
in the feature/gsoc-calc-perf branch which will eventually be merged to
master. If you are only changing the UI, I don't think there will be
any conflicts, but can you please show us your patch before you push? I
haven't committed my own changes to my feature branch yet, but I will soon.
Respectfully,
Daniel Bankston
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
