Edgewall Software
Modify

Opened 15 years ago

Closed 9 years ago

Last modified 9 years ago

#8429 closed defect (wontfix)

Path completion in "Prepare Diff" doesn't work with Internet Explorer

Reported by: Giacomo Tenaglia <Giacomo.Tenaglia@…> Owned by:
Priority: normal Milestone:
Component: version control Version: 0.11.3
Severity: normal Keywords: anydiff iexplorer suggest.js
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Hi, the dropdown menu doesn't behave correctly with IE (I've tested it with version 7 and 8): I can see the entries, but when I click on the scrollbar the dropdown disappears.

Cheers, Giacomo

Attachments (0)

Change History (6)

comment:1 by Christian Boos, 15 years ago

Keywords: anydiff iexplorer added
Milestone: 0.12.1

comment:2 by Ryan J Ollos, 9 years ago

Milestone: next-minor-0.12.xnext-stable-1.0.x

comment:3 by Ryan J Ollos, 9 years ago

Keywords: suggest.js added

comment:4 by Ryan J Ollos, 9 years ago

Milestone: next-stable-1.0.x
Resolution: wontfix
Status: newclosed

Closing as wontfix for 1.0-stable. suggest.js is replaced with jQuery-UI autocomplete for Trac 1.2 in #12169.

comment:5 by Jun Omae, 9 years ago

For Internet Explorer, clicking scrollbar of suggestion list leads blur event of input field.

  • trac/htdocs/js/suggest.js

    diff --git a/trac/htdocs/js/suggest.js b/trac/htdocs/js/suggest.js
    index 7559f6968..cba2468b9 100644
    a b  
    5454      }
    5555    });
    5656    input.blur(function() {
    57       if (timeout) clearTimeout(timeout);
    58       timeout = setTimeout(hide, 200);
     57      if (results && results[0] == document.activeElement)
     58        input[0].focus();
     59      else {
     60        if (timeout) clearTimeout(timeout);
     61        timeout = setTimeout(hide, 200);
     62      }
    5963    });
    6064
    6165    function hide() {
     
    7175
    7276    function move(index) {
    7377      if (!results) return;
    74       items = $("li", results);
     78      var items = $("li", results);
    7579      items.removeClass("selected");
    7680      $(items[index]).addClass("selected");
    7781      selectedIndex = index;
     
    125129          }
    126130        }
    127131        results.html(data).fadeTo("fast", 0.92);
    128         items = $("li", results);
     132        var items = $("li", results);
    129133        items
    130134          .hover(function() { move(items.index(this)) },
    131135                 function() { $(this).removeClass("selected") })

comment:6 by Ryan J Ollos, 9 years ago

It would probably be good to fix on 1.0-stable then since you have a patch. I just closed because it's 6 years old, and I figured the issue would never be addressed.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The ticket will remain with no owner.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none) to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.