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:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tvn created an issue. See original summary.

markhalliwell’s picture

I'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.

tim.plunkett’s picture

The "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.

David_Rothstein’s picture

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).

That 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 :)

yched’s picture

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.

+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 ++

joelpittet’s picture

+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

drumm’s picture

Assigned: Unassigned » drumm
attiks’s picture

+1 to move both "Reported by" and "Created" to a fixed place

kattekrab’s picture

Please put reported by and posted date back where they were.

I see no adequate rationale for their removal in any of the related issues.

markhalliwell’s picture

FileSize
35.35 KB

Sigh.. 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:

The "reported by" can stay in the 1st comment, that's fine.

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:

attiks’s picture

#10 I like the proposal, thanks

pfrenssen’s picture

I 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?

kattekrab’s picture

Thanks @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.

markhalliwell’s picture

No, 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:

I 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!

See my comment on: #2449497-26: Don't display author in 'Posted by..' line on issues

joshuami’s picture

The 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. :)

joshuami’s picture

kattekrab’s picture

I 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:

"The plan to resolve this—as of right now—is to add an automatic initial comment for the issue author. I'm hoping to get @drumm on this when he is back from vacation next week."

MustangGB’s picture

If 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.

dddave’s picture

From 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. ;)

markhalliwell’s picture

@kattekrab, from tvn (on #2446889-15: Display 'new' indicator next to user picture of new users):

Opened #2449489: Automatically generate comment when an issue is created for the automated comment idea

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.

markhalliwell’s picture

Issue summary: View changes

Add the mockup to the IS

yched’s picture

nitpick 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]

markhalliwell’s picture

Yes, in retrospect I could have ordered it a bit better. As I stated in #14 though:

This was just a quick mock-up to try and focus this "author-gate" from becoming more escalated than it already was.

I imagine that the ordering can easily be adjusted by @drumm as he sees fit.

drumm’s picture

drumm’s picture

FileSize
33.23 KB
dddave’s picture

Looking good from my perspective.

MustangGB’s picture

Feels 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?

pfrenssen’s picture

"Reporter" would be most appropriate I guess.

webchick’s picture

Category: Feature request » Task
Priority: Normal » Critical
Status: Active » Needs review

This is pretty critical information that just suddenly vanished.

#25 looks fine to me. We can always touch it up more in follow-ups.

drumm’s picture

I tried out "created by" to see if limiting to Drupal vocabulary worked. I agree, it isn't quite right. Reporter should work well enough.

  • drumm committed 2f78a1e on 7.x-3.x, dev
    Issue #2553355: Add "Reported by" and "Created" date to the sidebar on...
  • drumm committed 90324f0 on 7.x-3.x, dev
    Issue #2553355: Add "Reported by" and "Created" date to the sidebar on...
drumm’s picture

Status: Needs review » Fixed
Issue tags: +needs drupal.org deployment

Features 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.

drumm’s picture

Issue tags: -needs drupal.org deployment

Now deployed.

attiks’s picture

Thanks all

pfrenssen’s picture

I like it, nice work, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.