Making WordPress.org

Opened 6 years ago

Closed 6 years ago

#3839 closed enhancement (fixed)

Improve new meetup tracker log section

Reported by: sippis's profile sippis Owned by: sippis's profile sippis
Milestone: Priority: normal
Component: WordCamp Site & Plugins Keywords: good-first-bug has-patch needs-testing has-screenshots
Cc:

Description

Let me say first how I love the possibilities that new meetup tracker gives!

The log section gets a bit full of everything, as it does show notes and automatic messages like status and tag changes. It's good that there is a list of all things happened with the application, but sometimes deputies need to just check the note logs and see what has been previously said (vetting notes, 2nd opinions, etc etc).

For better separation of notes and automated log messages, I suggest the following changes:

  • add class for note log entries
  • set style rule color: #555 for #wcpt_log .widefat td, .widefat th
  • set style rule color: #222 for #wcpt_log .widefat tr.note td, .widefat tr.note th
  • apply function wpautop for notes, right now reading long notes is uncomfortable

Attached is a screenshot how log would be after applying these changes.

This a kind of a quick fix - in a long run I personally would like to see an option to show just notes if filter is selected.

Attachments (3)

Screenshot 2018-09-27 15.51.23.png (145.1 KB) - added by sippis 6 years ago.
Screenshot 2018-09-27 15.54.55.png (145.4 KB) - added by sippis 6 years ago.
Correct screenshot, first showed one automated messagw with styles intended for only notes
3839.diff (2.9 KB) - added by sippis 6 years ago.

Download all attachments as: .zip

Change History (9)

@sippis
6 years ago

Correct screenshot, first showed one automated messagw with styles intended for only notes

This ticket was mentioned in Slack in #meta-wordcamp by sippis. View the logs.


6 years ago

#2 @iandunn
6 years ago

  • Keywords needs-patch good-first-bug added; ui-feedback ux-feedback removed
  • Owner set to sippis
  • Status changed from new to assigned

@sippis
6 years ago

#3 @sippis
6 years ago

  • Keywords has-patch needs-testing has-screenshots added; needs-patch removed

Added a patch which implements these changes. Expected to look like this.

https://screenshotscdn.firefoxusercontent.com/images/c3a71f98-a8d4-45c2-8b4b-2039ec30635c.png

This ticket was mentioned in Slack in #meta-wordcamp by sippis. View the logs.


6 years ago

#5 @hlashbrooke
6 years ago

This is great - will definitely help a lot, especially adding wpautop to the note content!

Would be great if we could also automatically add a tags to and URLs in the note content (if wpautop doesn't do that already?), but even without that this is a great improvement!

#6 @vedjain
6 years ago

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

This was fixed via r8080

Note: See TracTickets for help on using tickets.