Last modified: 2013-10-14 17:08:21 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 T49256, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47256 - Consider installing Bugzilla's InlineHistory extension
Consider installing Bugzilla's InlineHistory extension
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Bugzilla (Other open bugs)
wmf-deployment
All All
: Normal enhancement (vote)
: ---
Assigned To: Andre Klapper
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-15 19:42 UTC by Andre Klapper
Modified: 2013-10-14 17:08 UTC (History)
8 users (show)

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


Attachments
Screenshot from Mozilla Bugzilla (45.45 KB, image/png)
2013-07-30 13:38 UTC, Andre Klapper
Details

Description Andre Klapper 2013-04-15 19:42:35 UTC
As this came up again today in a meeting with greg (switching between show_history and show_bug is cumbersome):

Can get all displayed in one by testing
http://bzr.mozilla.org/bugzilla/extensions/InlineHistory/
Comment 1 Gerrit Notification Bot 2013-07-26 15:14:47 UTC
Change 76105 had a related patch set uploaded by Aklapper:
Add bmo extension InlineHistory

https://gerrit.wikimedia.org/r/76105
Comment 2 Andre Klapper 2013-07-26 15:15:25 UTC
This extension has been in use on bugzilla.mozilla.org for two years, see https://bugzilla.mozilla.org/show_bug.cgi?id=668506

Its bug reports have been actively handled:
https://bugzilla.mozilla.org/buglist.cgi?&component=Extensions%3A InlineHistory&product=bugzilla.mozilla.org

https://bugzilla.mozilla.org/show_bug.cgi?id=668506#c20 states that "it _should_ work on other installations, however it's largely untested."
However, the extension is also in use on https://bugzilla.redhat.com .
Comment 3 Andre Klapper 2013-07-26 15:17:23 UTC
Furthermore, it creates a setting under userprefs.cgi called
    "When viewing a bug, show all bug activity"
which admins can turn on by default for logged-in users on editsettings.cgi.
Comment 4 Sumana Harihareswara 2013-07-30 13:25:13 UTC
From https://bugzilla.mozilla.org/show_bug.cgi?id=668506 as an example, the inline history only shows up if you're logged into Bugzilla.  Is that the preferred behavior?
Comment 5 Andre Klapper 2013-07-30 13:38:14 UTC
Created attachment 13019 [details]
Screenshot from Mozilla Bugzilla

Example screenshot from comments on a bug report in bugzilla.mozilla.org.
The parts that are shown by the InlineHistory extension are marked.
Note: This is only shown to users that are logged in.
Comment 6 Andre Klapper 2013-07-30 14:21:22 UTC
(In reply to comment #4)
> From https://bugzilla.mozilla.org/show_bug.cgi?id=668506 as an example, the
> inline history only shows up if you're logged into Bugzilla.  Is that the
> preferred behavior?

There is no setting for this, and one potential reason for not showing InlineHistory to non-logged-in users might be a missing implementation for obfuscating email addresses shown when changing e.g. the assignee of a report.
Comment 7 Alex Monk 2013-07-30 15:00:24 UTC
(In reply to comment #6)
> There is no setting for this, and one potential reason for not showing
> InlineHistory to non-logged-in users might be a missing implementation for
> obfuscating email addresses shown when changing e.g. the assignee of a
> report.

When I go to https://bugzilla.wikimedia.org/show_activity.cgi?id=1 logged out I just see the username part of the email address, without the domain. Can't that be done here?
Comment 8 Andre Klapper 2013-08-26 17:46:04 UTC
Deployed patch for testing at http://boogs.wmflabs.org/show_bug.cgi?id=2 - log in as horst_horstmann@gmx.net, use the part in front of @ as pass, and comment away.

Looking at it I prefer small CSS tweaks, e.g removing these two lines:
   .bz_comment.ih_history {
       padding: 5px 5px 0px 5px
   }
   .ih_inlinehistory {
       border-top: 1px dotted #C8C8BA;
   }
from style.css.
Comment 9 Andre Klapper 2013-08-28 10:01:39 UTC
Ah well, changes in comment 8 should be a separate patch after getting (unchanged compared to upstream) https://gerrit.wikimedia.org/r/76105 in, so it's easier to maintain.
Comment 10 Gerrit Notification Bot 2013-09-25 11:01:00 UTC
Change 76105 merged by Dzahn:
Add bmo extension InlineHistory

https://gerrit.wikimedia.org/r/76105
Comment 11 Andre Klapper 2013-09-25 11:08:29 UTC
Change deployed (thanks to dzahn).

To see this in action, go to https://bugzilla.wikimedia.org/userprefs.cgi?tab=settings and set "When viewing a bug, show all bug activity" to "On" (it's "Off" by default).

Now wondering whether to set this to "On" by default for all users under https://bugzilla.wikimedia.org/editsettings.cgi . I will probably do this soon, but I'd like to improve the CSS a bit before.
Comment 12 Gerrit Notification Bot 2013-09-25 13:15:08 UTC
Change 85978 had a related patch set uploaded by Aklapper:
Indent data added by InlineHistory so it is still easy to skim through normal comments. Followup to Change 76105.

https://gerrit.wikimedia.org/r/85978
Comment 13 Gerrit Notification Bot 2013-09-25 13:18:11 UTC
Change 85978 merged by Dzahn:
Indent data added by InlineHistory so it is still easy to skim through normal comments. Followup to Change 76105.

https://gerrit.wikimedia.org/r/85978
Comment 14 Daniel Zahn 2013-09-25 13:19:36 UTC
merged/deployed CSS fix
Comment 15 Andre Klapper 2013-09-25 15:54:31 UTC
All code changes are deployed so this is available now (see comment 11). 
Only question left is whether to enable this by default.
Comment 16 Andre Klapper 2013-09-30 15:03:57 UTC
Marking as FIXED for the time being (and keeping it as an opt-in).

I've announced availability on wikitech-l: http://lists.wikimedia.org/pipermail/wikitech-l/2013-September/072107.html
Comment 17 Andre Klapper 2013-10-14 17:08:21 UTC
(In reply to comment #15)
> Only question left is whether to enable this by default.

"Inline history" functionality is enabled by default now for every user who is logged into Bugzilla, as per feedback on http://lists.wikimedia.org/pipermail/wikitech-l/2013-October/072413.html

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


Navigation
Links