Ticket #8188 (closed defect (bug): fixed)

Opened 3 years ago

Last modified 3 years ago

Update Hello Dolly

Reported by: janeforshort Owned by:
Priority: low Milestone: 2.7
Component: UI Version:
Severity: minor Keywords:
Cc:

Description

Reduce font size, position just to the left of screen options tab.

Attachments

hello.patch Download (550 bytes) - added by zedlander 3 years ago.
hello.png Download (7.1 KB) - added by zedlander 3 years ago.
Screenshot
8188.r9683.diff Download (1.5 KB) - added by jacobsantos 3 years ago.
Better standards for example plugin, includes style patch.

Change History

  • Type changed from defect to enhancement
  • Milestone changed from 2.7 to 2.8

No patch, looks like an enhancement.

  • Type changed from enhancement to defect

Color sucks too -> Change it from red to something that doesn't look bad. Like Green.

comment:3   ryan3 years ago

  • Milestone changed from 2.8 to 2.7

We have to do something with it in 2.7 cause right now it sucks. I'd prefer to remove it, but some people want to keep the useless thing.

Some people... like your boss? :)

I want it removed too. The purpose is to demonstrate how to make a WP plugin and while it provides a functional example of a WP plugin, a more involved plugin would be useful. One that adds, for example, a management page, uses a hook or two (including one that takes more than two arguments, for instance) and one that is built in an object oriented way to demonstrate preferred coding techniques for a WP plugin.

I'm just saying that Hello Dolly, while a little useful from a demonstration perspective, really could use more demonstration functionality. Rewrite Hello Dolly in a more complex way, and it would still be functionally useless but demonstrably more useful. :)

comment:5   DD323 years ago

My understanding was that Hello Dolly is supposed to be used as an example plugin, But like technosailor says, Its an utter aweful example of a plugin, Global values, running code in non-function/hook aspect, etc. It really would be good to replace it with something else which DOES demonstrate a lot more functionality..

Screenshot

How does this look? hello.patch and hello.png attached.

Better standards for example plugin, includes style patch.

Looks hot.

comment:8   ryan3 years ago

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

(In [9684]) Hello Dolly update. Props zedlander and jacobsantos. fixes #8188

Note: See TracTickets for help on using tickets.