Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The change over at #2449497: Don't display author in 'Posted by..' line on issues removed information about original issue author and created date from old issue nodes completely. This information is important and needs to be present on the issue page in some form.
Proposed resolution
Add "Reported by" and "Created" date to the sidebar metadata block on the issue pages:
Comment | File | Size | Author |
---|
Comments
Comment #2
markhalliwellI'm confused. What was the point of #2449489: Automatically generate comment when an issue is created then?
I thought we agreed in #2449497: Don't display author in 'Posted by..' line on issues that if people needed to see this information on older issues they could look at the revisions (which tells you the same information).
Regardless, this sounds like duplicating information, one or the other needs to go IMO.
Personally, I'd be fine with this showing up in the sidebar only (with pic and new status/confirm button) and then we just get rid of the automated comment.
Comment #3
tim.plunkettThe "reported by" can stay in the 1st comment, that's fine.
But an IS can be very very long, and having the created date in a fixed/reliable place is VERY important.
Right above the "Last updated" in the sidebar is a great spot.
Comment #4
David_Rothstein CreditAttribution: David_Rothstein as a volunteer commentedThat only works if you're logged in, though (and if you know to look for it there).
I think this is pretty important information and that both things ("reported by" and "created date") definitely belong somewhere on the main page where everyone can see them. I noticed their absence pretty quickly while reviewing some older issues today :)
Comment #5
yched CreditAttribution: yched commented+1, it's really unfortunate to have to scroll an undefinite amount after the IS to find the creation date. Not sure why the sidebar would be a better place than where it used to be until recently (below the title), but fixed place ++
Comment #6
joelpittet+1 this issue for "Reported By", I can't tell by looking at the page if I even posted the original IS. I use this all the time because my writing style isn't unique enough:P
Edit: Who posted this issue? lol
Comment #7
drummComment #8
attiks CreditAttribution: attiks commented+1 to move both "Reported by" and "Created" to a fixed place
Comment #9
kattekrab CreditAttribution: kattekrab at Creative Contingencies commentedPlease put reported by and posted date back where they were.
I see no adequate rationale for their removal in any of the related issues.
Comment #10
markhalliwellSigh.. ok, since this is escalating rather quickly and since no one has been following the original line of events that has lead to this issue, here's a little back-history:
I soon closed my follow-up in favor of this issue because having two locations for this type of information is ridiculous. This initial comment should just be removed. I never really liked the idea to begin with, but didn't have very valid reasons to speak up against the idea. It was an idea that was attempting to solve an issue, ironically it created more than it was worth IMO.
Regardless, I think all this information should just be moved to the sidebar where it really belongs to begin with. @drumm, here's a mockup:
Comment #11
attiks CreditAttribution: attiks commented#10 I like the proposal, thanks
Comment #12
pfrenssenI think we should highlight the original author of the issue summary, not hide them away in the sidebar. It takes an effort for people to sit down and report an issue, we should encourage that!
Why not style it like the comments are, with the profile picture to the left and the issue summary to the right?
Comment #13
kattekrab CreditAttribution: kattekrab at Creative Contingencies commentedThanks @markcarver - that's an elegant alternative.
However - I think #2288727: [meta] Provide credit to organizations / customers who contribute to Drupal issues is what started this
moreso than #2446889: Display 'new' indicator next to user picture of new users
There's also #2474609: Not possible to credit people who didn't comment in an issue
So, in balance - I think we've made some cludgey work arounds which created new problems that didn't exist before hand.
And "people don't know they can edit issue summaries" seems like a post-hoc rationalisation.
That said, I reckon @markcarver's mockup here is an actual improvement. I feel like I want to swap the order of "last updated" and "created" though - so that it's adjacent to reported by, and hierarchically, I feel "last updated" should have precedence.
Comment #14
markhalliwellNo, the credit stuff did not initiate this chain of events (https://www.drupal.org/node/2446889#comment-9704249).
Re: #2474609: Not possible to credit people who didn't comment in an issue
FWIW, we shouldn't have to have a comment to add the OP to the list of commit credits (this can be added in programmatically). This is a separate problem entirely.
Re: swap the order of "last updated" and "created"
That's fine, I really don't care what order they're in. This was just a quick mock-up to try and focus this "author-gate" from becoming more escalated than it already was.
edit:
See my comment on: #2449497-26: Don't display author in 'Posted by..' line on issues
Comment #15
joshuamiThe initial comment is not required for #2474609: Not possible to credit people who didn't comment in an issue but it does help solve that issue and provide the confirm button in a handy place as pointed out in #2474609: Not possible to credit people who didn't comment in an issue.
+1 to the mockup from @markcarver. That was the intent, so its nice to see it as preview. I like the addition of the "new" tag next to the user name. If that is not too hard to implement, it definitely provides good info to the maintainer that will help temper any response—hopefully :)
Also +1 to calling this even "author-gate". That made my day. :)
Comment #16
joshuamiComment #17
kattekrab CreditAttribution: kattekrab at Creative Contingencies commentedI just found the comment by @joshuami that I reckon started this chain reaction.
#2491115: As a project maintainer, I cannot save credit for the original poster in an issue
In particular https://www.drupal.org/node/2491115#comment-9948783
@joshuami said:
Comment #18
MustangGB CreditAttribution: MustangGB commentedIf it's of any UX help to you after noticing this information had disappeared then discovering it didn't re-appearing on page refresh I assumed it was a d.o customization and presumed it would have been moved to the sidebar metadata.
I didn't even notice the additional first comment until hunting the issue queue for entries relating to this change and reading about it.
Comment #19
dddave CreditAttribution: dddave commentedFrom a daily webmastering perspective having the original author at display without having to check revisions for older issues would be very welcome. Same goes for the original date of creation. This annoyed me quite a bit today. ;)
Comment #20
markhalliwell@kattekrab, from tvn (on #2446889-15: Display 'new' indicator next to user picture of new users):
This was in direct correlation to the "new" indicator. The commit stuff just jumped on board with this idea. TBH though, a comment is not necessary to add the OP (issue author/reporter) to the generated list of commit credit. They can easily be injected programmatically since the issue author is part of the node when this list is being generated. We don't need the initial "first comment", it is redundant and just clutters things.
Comment #21
markhalliwellAdd the mockup to the IS
Comment #22
yched CreditAttribution: yched commentednitpick on the mockup :
Created : [date]
Last updated : [date]
Reported by : [user]
is not a very intuitive order ("Created" and "By" go together, conceptually)
Why not :
Reported by : [user]
On : [date]
Last updated : [date]
or
Created : [date]
By : [user]
Last updated : [date]
Comment #23
markhalliwellYes, in retrospect I could have ordered it a bit better. As I stated in #14 though:
I imagine that the ordering can easily be adjusted by @drumm as he sees fit.
Comment #24
drummHere it what it looks like so far at https://ci-drupal.redesign.devdrupal.org/node/2547851
Comment #25
drummAnd with a new marker: https://ci-drupal.redesign.devdrupal.org/node/2546440
Comment #26
dddave CreditAttribution: dddave commentedLooking good from my perspective.
Comment #27
MustangGB CreditAttribution: MustangGB commentedFeels a bit weird to have "Created by", I mean we don't have "Assigned to", "Created on", or "Updated on", also it's a bit too similar to the plain old "Created", is there a reason for not having a single word description, for example "Author", "Reporter", "Raiser", "Creator", "Owner", "Originator", or something else along those lines?
Comment #28
pfrenssen"Reporter" would be most appropriate I guess.
Comment #29
webchickThis is pretty critical information that just suddenly vanished.
#25 looks fine to me. We can always touch it up more in follow-ups.
Comment #30
drummI tried out "created by" to see if limiting to Drupal vocabulary worked. I agree, it isn't quite right. Reporter should work well enough.
Comment #32
drummFeatures doesn't seem to pick up extra display fields, which I added in project_issue with #2553961: Add extra fields for reporter, created time, and updated time. On deployment, this will briefly show the new metadata also in the main content area. Configuring these via Field UI will be a field cache clear or three, so you may notice instability, or at least long page loads, if you time the requests just right.
Comment #33
drummNow deployed.
Comment #34
attiks CreditAttribution: attiks commentedThanks all
Comment #35
pfrenssenI like it, nice work, thanks!