Making WordPress.org

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#1268 closed enhancement (wontfix)

DevHub: Hide the filenames in the Used By/Uses section

Reported by: drewapicture's profile DrewAPicture Owned by:
Milestone: Priority: normal
Component: Developer Hub Keywords: good-first-bug has-patch
Cc:

Description (last modified by DrewAPicture)

I'd like to propose hiding the filenames in the "Used By/Uses" section on code reference pages. I think it largely serves to cause unnecessary wrapping within the container, and realistically, I feel like developers would be more concerned with the functions the current element is used in rather than both the file and the function.

Besides, the linked function reference would list the source file anyway.

For example, this is the "Used By" section on the plugins_api() reference currently:

http://f.cl.ly/items/1c3S153v2n252O3g0M0R/Screen%20Shot%202015-09-29%20at%2010.10.52%20AM.png

I think simply displaying the functions would make this a lot cleaner.

Attachments (1)

1268.patch (1.6 KB) - added by chris_dev 9 years ago.
I change anchor tags to show the filename in a tooltip base on the opinion from @jancbeck

Download all attachments as: .zip

Change History (8)

#1 @DrewAPicture
9 years ago

  • Summary changed from DevHub: Hide the filenames in the Used By section to DevHub: Hide the filenames in the Used By/Uses section

#2 @DrewAPicture
9 years ago

  • Description modified (diff)

#3 @Otto42
9 years ago

I like having the filenames there, but the wrapping is annoying. Maybe make the font smaller?

#4 @jancbeck
9 years ago

I suggest hiding the filenames but including them via title="wp-admin/...". That way they appear when hovering the link.

@chris_dev
9 years ago

I change anchor tags to show the filename in a tooltip base on the opinion from @jancbeck

#5 @chris_dev
9 years ago

  • Keywords has-patch added; needs-patch removed

#6 follow-up: @DrewAPicture
9 years ago

  • Resolution set to wontfix
  • Status changed from new to closed

The consensus from @Otto42 here and @coffee2code elsewhere suggests this is a wontfix.

@chris_dev @jancbeck Either way, we would not want to leverage a title attribute for extra information about the files themselves. It doesn't follow best-practices for accessibility. Regardless, thank you for the patch and discussion

#7 in reply to: ↑ 6 @chris_dev
9 years ago

Understood :)

Replying to DrewAPicture:

The consensus from @Otto42 here and @coffee2code elsewhere suggests this is a wontfix.

@chris_dev @jancbeck Either way, we would not want to leverage a title attribute for extra information about the files themselves. It doesn't follow best-practices for accessibility. Regardless, thank you for the patch and discussion

Note: See TracTickets for help on using tickets.