Opened 2 years ago

Last modified 2 years ago

#16280 new enhancement

UI improvements to Themes API

Reported by: solarissmoke Owned by:
Priority: normal Milestone: Awaiting Review
Component: Themes Version: 3.1
Severity: minor Keywords:
Cc:

Description

Following on from #16132, some improvements to the Themes API UI. Here's a patch for one particular issue, but there may be more overhaul for WP 3.2

Attachments (1)

16280.diff​ (665 bytes) - added by solarissmoke 2 years ago.
Pass on original wp_error when http request fails

Download all attachments as: .zip

Change History (4)

solarissmoke — 2 years ago

Pass on original wp_error when http request fails

comment:1 follow-up: ↓ 2   nacin — 2 years ago

I don't like this patch. As it is, the error message we're sending does very little to explain what went wrong. Providing the raw HTTP error might only make it worse.

These errors, strings, and UX need a real overhaul in 3.2.

comment:2 in reply to: ↑ 1   solarissmoke — 2 years ago

Replying to nacin:

I don't like this patch. As it is, the error message we're sending does very little to explain what went wrong. Providing the raw HTTP error might only make it worse.

IMO "An unexpected HTTP error occurred" is far less helpful than the original HTTP error. But I agree that the whole thing needs improvement, so maybe minor tweaks isn't the way to go.

comment:3   nacin — 2 years ago

  • Type changed from defect (bug) to enhancement
Note: See TracTickets for help on using tickets.