Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


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.libreoffice@gmail.com <mailto:daniel.dev.libreoffice@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

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.