Last modified: 2013-02-21 16:48:40 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T37468, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 35468 - gerrit download instructions hide when you click them
gerrit download instructions hide when you click them
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Git/Gerrit (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: upstream
Depends on: 35466
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-25 14:43 UTC by Platonides
Modified: 2013-02-21 16:48 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Platonides 2012-03-25 14:43:31 UTC
Download instructions hide when you try to copy them

Steps to reproduce:
- Go to a changeset
- Click in the download section instead of selecting (mousedown + move)
- The download section with the patchset-specific instructions has disappeared. There's no option to expand it again.


https://android-review.googlesource.com/ converts that  it into a selected textbox, so so it might be fixed in a newer gerrit version.
Comment 1 Chad H. 2012-05-04 18:11:18 UTC
I believe this is fixed in 2.3. Can you please check and confirm?
Comment 2 Alex Monk 2012-05-04 18:35:29 UTC
It seems to convert into a textbox now. I was pretty sure it already did this, but oh well.
Comment 3 Platonides 2012-05-05 22:08:36 UTC
I'm viewing the textbox behavior on WebKit browsers, but still the hiding on Firefox.

I tried to bypass the cache, but it could still be playing tricks on me. Is all javascript inline?

Perhaps related, when I single click that region I get «uncaught exception: Class$LXb: One or more exceptions caught, see full set in UmbrellaException#getCauses»
Comment 4 Platonides 2012-05-19 13:57:43 UTC
Also reported by liangent
Comment 5 Chad H. 2012-06-13 21:28:18 UTC
Is http://code.google.com/p/gerrit/issues/detail?id=1176 what you're seeing?
Comment 6 Platonides 2012-06-13 22:14:42 UTC
Looks the same issue. What I don't understand is why it works in android-review.googlesource.com but fails on gerrit.wikimedia.org even after upgrading gerrit a couple of times.
Comment 7 Chad H. 2013-02-12 16:08:19 UTC
This is working fine for me now with the latest upgrade. How about you?
Comment 8 Chad H. 2013-02-14 16:10:03 UTC
Going to go ahead and mark this fixed...has been working reliably for me all week since we upgraded.
Comment 9 Platonides 2013-02-21 16:48:40 UTC
Seems fixed.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links