Tickets :: Comment on [#4560] Autocomplete broken in IE6

* Your Email Address
* Spam protection	Enter the letters below: . .. .. \ /. \|\ \|\| \|\| >< \| \| \\|\|__\|\|___/ \\|___
Comment	> But your code is breaking Auto Complete. Regardless of your or my > opinions on the Safety/Good Idea of using Auto Complete, it isn't > working because of code that was added to Imp's login.inc between > versions 3.2.1 and 4.1.2 of Imp. (Actually, we know from the > changelogs when you added that code in...but there wasn't much > information on why it's there or what it does) > > > > Anyway, you say that we can "comment out the code at our own risk" > > But you don't offer what I consider to be an adequate solution, but > instead offer reasons why we shouldn't want that functionality in the > first place (which is kind of an admission that it's broken, IMO, a > bug) --- If I have users that want that functionality, and I, as > system admin, am not turning it off, then shouldn't they have it if > they want? > > > > Again, regardless of the politics of Auto Complete, isn't the fact > that the addition to your code breaks browser functionality a bug? > And the removal of said piece of code makes it work again. How is > that not a bug? > > > > If the code is necessary for certain other functionality, as you say > in point 1, then maybe you should fix the broken code so people can > make their own decisions about the rest.
Attachment
Watch this ticket