Making WordPress.org

Opened 8 years ago

Closed 8 years ago

Last modified 10 months ago

#2359 closed enhancement (fixed)

Support Theme: Convert "[Resolved]" to a checkmark

Reported by: lukecavanagh's profile lukecavanagh Owned by: coffee2code's profile coffee2code
Milestone: Priority: normal
Component: Support Forums Keywords:
Cc:

Description

If you go to a plugins support page, there is not an easy way to be able to visually see which post have been resolved.

Attachments (3)

Support Theme Resolved Color Change.png (144.4 KB) - added by lukecavanagh 8 years ago.
Support Example
meta-2359_topic-view_plain.png (185.6 KB) - added by coffee2code 8 years ago.
Prepending of plain green checkmark before topic title in single topic view.
meta-2359_topic-view.png (188.9 KB) - added by coffee2code 8 years ago.
More prominent green checkmark badge of sorts before topic title in single topic view.

Download all attachments as: .zip

Change History (23)

#1 @samuelsidler
8 years ago

Each topic title has "[Resolved]" in front of it, which is a good visual indicator that it has been resolved.

I'm not keen on highlighting resolved topics with a different color or otherwise because I don't know for which audience that would be useful. Making resolved topics stand out makes the unresolved ones less visible when scanning the list. Do plugin authors want resolved topics to be highlighted? Why?

#2 follow-up: @jimtrue
8 years ago

Some sort of visual indicator (that is more than just the 'word' 'Resolved') would be helpful, yes. The old system did this before the forum theme was changed. It was one of the reasons it was so easy to 'work the support queue' and hit all the tickets that hadn't been either responded to or resolved. An icon (that had an ARIA value) would be helpful. You don't have to show them to all 'users' of the forums, just the ones that are Admins of the plugin in question.

#3 in reply to: ↑ 2 @samuelsidler
8 years ago

  • Summary changed from Support Theme: Showing resolved posts in a different color to Support Theme: Convert "[Resolved]" to a checkmark

Replying to jimtrue:

Some sort of visual indicator (that is more than just the 'word' 'Resolved') would be helpful, yes.

Got it. @mapk had a checkmark in the mockups (can you post those?), but we ran into some technical issues implementing it. @coffee2code, can you help out here? We'd want to convert "[Resolved]" to a checkmark with the appropriate ARIA value.

#4 @mapk
8 years ago

Ignore some of the other stuff, but you can see the checkmark there along side the topic title. Pretty simple, and follows format of the other icons like the pin.

https://cldup.com/ZjygD_TZPY.png

#5 @coffee2code
8 years ago

  • Owner set to coffee2code
  • Status changed from new to accepted

@coffee2code
8 years ago

Prepending of plain green checkmark before topic title in single topic view.

@coffee2code
8 years ago

More prominent green checkmark badge of sorts before topic title in single topic view.

#6 follow-up: @coffee2code
8 years ago

I have this just about ready, but no consideration had been given to single topic views, where "[RESOLVED]" was also being prepended to the topic title. Three suggestions for that:

  1. Leave as-is and prepend "[RESOLVED]" before titles of resolved topics.
  2. Prepend a plain green checkmark as done in the topic listings. See meta-2359_topic-view_plain.png.
  3. Give the green checkmark a bit more prominence and flair. See meta-2359_topic-view.png. (Not shown: at smaller viewport widths the checkmark reverts to the "plain" style so as not to appear half off-screen.)

Thoughts?

/cc @samuelsidler

#7 in reply to: ↑ 6 @mapk
8 years ago

Replying to coffee2code:

I have this just about ready, but no consideration had been given to single topic views, where "[RESOLVED]" was also being prepended to the topic title.

I actually had a mockup for that as well. Here it is.

https://cldup.com/O7RwlNNLaN.png

#8 @coffee2code
8 years ago

@mapk: Looks good! That should work.

This ticket was mentioned in Slack in #bbpress by netweb. View the logs.


8 years ago

#10 @johnjamesjacoby
8 years ago

These Star Wars themed conversations are hilarious.

#11 follow-up: @coffee2code
8 years ago

In 4639:

Support Forums, Topic Resolution: Denote resolved topics in better ways than prepending "[RESOLVED]" to topic titles.

  • Prepends markup to resolved topic titles in topic listings, suitable for being shown as a checkmark.
  • Outputs resolved indicator in single topic view for resolved topics, with the text "Answered".
  • Discontinues prepending "[RESOLVED]" to topic titles.

Props mapk, coffee2code.
See #2359.

#12 @jimtrue
8 years ago

Wow, Great work @coffee2code and @mapk Looks great and much cleaner!

#13 @coffee2code
8 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 4640:

Support Theme: Prepend titles of resolved topics with green dashicon checkmark instead of "[RESOLVED]" in topic listings.

Also styles resolved indicator in single topic views.

Props mapk, coffee2code.
Fixes #2359.

#14 in reply to: ↑ 11 ; follow-up: @SergeyBiryukov
8 years ago

Replying to coffee2code:

  • Outputs resolved indicator in single topic view for resolved topics, with the text "Answered".

Is there a reason not to use "Resolved" there? An answered topic doesn't necessarily mean it's resolved, using a consistent term would less confusing.

#15 in reply to: ↑ 14 ; follow-up: @mapk
8 years ago

Replying to SergeyBiryukov:

Replying to coffee2code:

  • Outputs resolved indicator in single topic view for resolved topics, with the text "Answered".

Is there a reason not to use "Resolved" there? An answered topic doesn't necessarily mean it's resolved, using a consistent term would less confusing.

Very true. Let's change the wording to 'resolved' in the new design.

#16 in reply to: ↑ 15 @SergeyBiryukov
8 years ago

Replying to mapk:

Very true. Let's change the wording to 'resolved' in the new design.

Thanks! Created #2380 for that.

#17 @coffee2code
8 years ago

In 4644:

Support Theme: Ensure resolved topic title checkmark displays in user replies view.

Also minor tweaks to checkmark styles.

See #2359.

#18 @coffee2code
8 years ago

In 4647:

Forums Theme: Backport [4639], which denotes resolved topics in better ways than prepending "[RESOLVED]" to topic titles.

Slight, but inconsequential, differences to the resolved indicator on single topic views.

See #2359.

#19 @SergeyBiryukov
8 years ago

In 5020:

Support Theme: Correct resolved topic checkmark position in Spam/Pending/Archived views and breadcrumbs.

See #2359.

#20 @SergeyBiryukov
8 years ago

In 5062:

Support Theme: Revert [5020] and use another approach for resolved topic checkmark styling.

This keeps visual appearance the same, but removes the need for display: inline-block workaround, which caused unnecessary wrapping with longer titles.

See #2359.

Note: See TracTickets for help on using tickets.