Ask HN: I found an obscure bug with Firefox. Is it worth reporting?

5 pointsheigh1y ago16 comments

It’s really quite strange but when you use a “number” input field in Firefox, then validate it with RegEx, it totally flips out.

I can provide examples but I guess my question is, is it worth reporting these? Or will it just never get fixed?

Edit: spellcheck

16 comments

toast01y ago

IMHO, it's worth reporting, but temper your expectations of it being fixed. I've seen clearly written, easily reproduced, possibly easily fixed bugs linger in Bugzilla for decades. At Mozilla and other organizations using bugzilla and other trackers.

Spend a day turning this into a bug report that's up to the standards of Mozilla; maybe another day trying to track down the behavior and suggest a patch. File the bug, and maybe the patch, and then implement a work around and go on with your life. Sure, watch for replies and try to respond in a timely fashion, but don't expect anything.

That said, I've put in a bugzilla bug and it got fixed in a reasonable time and then pushed in the next eligible release; https://bugzilla.mozilla.org/show_bug.cgi?id=1740856 but it was widely applicable, and the observed behavior was clearly broken, but details were hard to observe for many (which is why effects from the bug were observed but not diagnosed in related bugs)

runjake1y ago

I am not super good at this and only using half my brain, but I did my best attempt to convert this to vanilla JavaScript and get seemingly odd results.

    <!DOCTYPE html>
    
    <input type="number" id="progress" class="form-control mr-0 pr-0" placeholder="Progress (%)" />
    
    <script>
    document.getElementById('progress').addEventListener('change', function() {
        console.log("fired!");

        // Remove any non-numeric.
        let val = this.value.replace(/[^0-9.]/g, '');
    
        // Only one decimal point.
        let parts = val.split('.');
        if (parts.length > 2) {
            val = parts[0] + '.' + parts.slice(1).join('');
        }
    
        let numVal = parseFloat(val) || 0;
    
        if (numVal > 100) {
            numVal = 100;
        } else if (numVal < 0) {
            numVal = 0;
        }
    
        this.value = numVal === 0 ? '' : numVal;
    });
    </script>

Someone educate me, especially if I am off base or the code is inherently incorrect.

heighOP1y ago

Scrapping the jQuery is definitely useful to recreate it, thanks for that. The only thing I would do extra is adding in HTML, HEAD and BODY tags to make it a bit more syntactically correct...

Thanks for this though!

BoredPositron1y ago

If you are going to report it like you wrote this post, you don't need to report it.

galdosdi1y ago

Agreed. Do a lot more investigation, including creating a minimal test case, carefully reading MDN to be sure the behavior isn't expected (and you should parse the docs to be able to state precisely what the intended behavior and output should be), and honestly, give a shot at actually debugging a development version of Firefox itself. You may well fix the bug yourself (and in the process learn a lot, or more likely realize your bug is not a bug and you just misunderstood the API)

You really want to avoid wasting people's time with a nonbug that is due to you misunderstanding a subtle API.

This is open source. There is no them. Them is us. The easier you make the bug ticket to read the likelier someone chooses to try to fix it.

I mean you do this for a living too so think of it like that -- are you writing a bug ticket that screams "nice easy junior beginner bug" or something that screams "I'm gonna investigate forever and never even figure out why the reporter thinks this is an issue"?

If you can make a truly good bug report it actually does have a good chance of being fixed, if not now, eventually, next time someone wants to "start contributing to open source" and sees this easy beginner bug ticket. Believe me, nobody will want to tackle one that's written like your above comment

Also, using a regex on a number doesn't sound that obscure. Ask yourself why you're the first one to catch this.

Also you must repro it on the latest nightly build or else you may be reporting something already fixed

Basically just do unto them as you would have your users do unto you in your day job projects.

heighOP1y ago

My sincere apologies to you both - I am mostly a lurker, and really didn't expect a response to my post, especially one as detailed as you've given.

I'm a 20 year dev, and it really is a strange bug, but as I mentioned before, I didn't expect a response, so as I only had my password for HN on my phone (which I only have stored there), I didn't have access to my code, nor thought to bother to reset my password and store it so I can access it from my PC.

I'm kicking my laziness/sloppiness-self now, and sorry for my crappy original post. I reset my password and want to thank you both for your feedback.

I will post the bug and see how it goes...

heighOP1y ago

For what it's worth the issue I encountered is this, and now, I am wondering if the ticket should actually be directed to jQuery, rather than Firefox...

The bug is actually around a type="number" input field, not a number in an INPUT field. But here goes my summary of it:

<input type="number" id="progress" class="form-control mr-0 pr-0" placeholder="Progress (%)" />

//JS Code $('#progress').on("change",function() { let val = $(this).val().replace(/[^0-9.]/g, '');

        // Ensure only one decimal point
        let parts = val.split('.');
        if (parts.length > 2) {
            val = parts[0] + '.' + parts.slice(1).join('');
        }

        let numVal = parseFloat(val) || 0;

        if (numVal > 100) {
            numVal = 100;
        } else if (numVal < 0) {
            numVal = 0;
        }

        $(this).val(numVal === 0 ? '' : numVal);
    });

//Expectation OnChange, it removed everything except valid numbers (including a single decimal place).

//Reality It does this for the first iteration, but not further iterations

//Example 1. Enter "1234.456AB" into the progress field (excluding the quotation marks). 2. blur from the field to trigger the onChange. 3. Observe that it works and produces the result 100 (as per the code). 4. Type in ABCD into the progress field 5. Nothing happens. ABCD remains there.

2 more replies

JohnFen1y ago

You're not completely wrong, but posing all of that as some sort of hard requirement also encourages people to just not report bugs.

There needs to be a balance, but I'm not sure what it would look like.

1 more reply

rossng1y ago

Yes, you should report it. I've reported several Firefox bugs and the tickets often spring to life suddenly after several years of sitting there unaddressed. It makes life much easier for the developers if you include a clean, easy-to-reproduce test case.

KomoD1y ago

Honestly, you could have spent the time you used to write this post and comments on making the report instead.

You basically wrote the entire bug report here, instead of doing it on Bugzilla.

j / k navigate · click thread line to collapse

16 comments

toast01y ago

runjake1y ago

I am not super good at this and only using half my brain, but I did my best attempt to convert this to vanilla JavaScript and get seemingly odd results.

    <!DOCTYPE html>
    
    <input type="number" id="progress" class="form-control mr-0 pr-0" placeholder="Progress (%)" />
    
    <script>
    document.getElementById('progress').addEventListener('change', function() {
        console.log("fired!");

        // Remove any non-numeric.
        let val = this.value.replace(/[^0-9.]/g, '');
    
        // Only one decimal point.
        let parts = val.split('.');
        if (parts.length > 2) {
            val = parts[0] + '.' + parts.slice(1).join('');
        }
    
        let numVal = parseFloat(val) || 0;
    
        if (numVal > 100) {
            numVal = 100;
        } else if (numVal < 0) {
            numVal = 0;
        }
    
        this.value = numVal === 0 ? '' : numVal;
    });
    </script>

Someone educate me, especially if I am off base or the code is inherently incorrect.

heighOP1y ago

Scrapping the jQuery is definitely useful to recreate it, thanks for that. The only thing I would do extra is adding in HTML, HEAD and BODY tags to make it a bit more syntactically correct...

Thanks for this though!

BoredPositron1y ago

If you are going to report it like you wrote this post, you don't need to report it.

galdosdi1y ago

You really want to avoid wasting people's time with a nonbug that is due to you misunderstanding a subtle API.

This is open source. There is no them. Them is us. The easier you make the bug ticket to read the likelier someone chooses to try to fix it.

Also, using a regex on a number doesn't sound that obscure. Ask yourself why you're the first one to catch this.

Also you must repro it on the latest nightly build or else you may be reporting something already fixed

Basically just do unto them as you would have your users do unto you in your day job projects.

heighOP1y ago

My sincere apologies to you both - I am mostly a lurker, and really didn't expect a response to my post, especially one as detailed as you've given.

I'm kicking my laziness/sloppiness-self now, and sorry for my crappy original post. I reset my password and want to thank you both for your feedback.

I will post the bug and see how it goes...

heighOP1y ago

For what it's worth the issue I encountered is this, and now, I am wondering if the ticket should actually be directed to jQuery, rather than Firefox...

The bug is actually around a type="number" input field, not a number in an INPUT field. But here goes my summary of it:

<input type="number" id="progress" class="form-control mr-0 pr-0" placeholder="Progress (%)" />

//JS Code $('#progress').on("change",function() { let val = $(this).val().replace(/[^0-9.]/g, '');

        // Ensure only one decimal point
        let parts = val.split('.');
        if (parts.length > 2) {
            val = parts[0] + '.' + parts.slice(1).join('');
        }

        let numVal = parseFloat(val) || 0;

        if (numVal > 100) {
            numVal = 100;
        } else if (numVal < 0) {
            numVal = 0;
        }

        $(this).val(numVal === 0 ? '' : numVal);
    });

//Expectation OnChange, it removed everything except valid numbers (including a single decimal place).

//Reality It does this for the first iteration, but not further iterations

2 more replies

JohnFen1y ago

You're not completely wrong, but posing all of that as some sort of hard requirement also encourages people to just not report bugs.

There needs to be a balance, but I'm not sure what it would look like.

1 more reply

rossng1y ago

KomoD1y ago

Honestly, you could have spent the time you used to write this post and comments on making the report instead.

You basically wrote the entire bug report here, instead of doing it on Bugzilla.

j / k navigate · click thread line to collapse