Skip to content
This repository has been archived by the owner on Sep 4, 2018. It is now read-only.

Adjustments to widget template #2

Closed
wants to merge 1 commit into from
Closed

Adjustments to widget template #2

wants to merge 1 commit into from

Conversation

mfields
Copy link

@mfields mfields commented Dec 10, 2011

  • Use WordPress "template" style for the widget template.
  • Escape all date before printing to the screen.
  • Update i18n to reflect entire sentence.

Adjustments to widget template:
346f3a0 
 * Use WordPress "template" style for the widget template.
 * Escape all date before printing to the screen.
 * Update i18n to reflect entire sentence.
@ericmann
Copy link
Owner

Great call on the i18n setup. And a better pattern than building and echoing a string, too. Though it can't be auto-merged, I will merge it once I get a free second.

@ericmann
Copy link
Owner

Manually merged :-)

@ericmann ericmann closed this Dec 11, 2011
@mfields
Copy link
Author

mfields commented Dec 14, 2011

THanks! Glad to see the changes made it in there :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mfields @ericmann