PATCH: Twitter settings page missing text in HTML title element

Submitted by meitar on Fri, 01/23/2009 - 12:41

Issue ID:	975
Issue Category:	bug
Component:	ui
Priority:	normal
Status:	fixed
Assigned:	zach
Version:	0.7
Milestone:	1.0
Keywords:	patch

When viewing one's Twitter settings page, the `title` element in the HTML output is missing some text. Specifically, the HTML code for the title is output as follows:

{{{
- Identi.ca
}}}

By comparison, see how the SMS Settings page title appears:

{{{
SMS Settings - Identi.ca
}}}

The title on the Twitter page therefore seems as though it should say, "Twitter Settings" or similar.

Legacy Data

This issue was migrated from another tracking system. The legacy data at time of import is provided below as a reference.

Ticket ID:

975

Reported by:

meitarm@gmail.com

Owner:

zcopley

Status:

closed

Resolution:

closed

Type:

bug

Component:

Priority:

Version:

0.7

Milestone:

1.0

Attachment	Size
975_twitter-settings-title.git.patch	365 bytes

Updates

#1

Submitted by meitar on Tue, 01/27/2009 - 14:20.

Easy fix; just a missing `return` statement. :)

#2

Submitted by Zach Copley on Tue, 01/27/2009 - 18:35.

That was easy! Thanks.

#3

Submitted by meitar on Wed, 01/28/2009 - 09:55.

I noticed that this patch was applied to the Laconica repo. What's the appropriate process this project uses to close tickets on Trac? I.e., if I see a patch of mine applied, like this one, can I return to the ticket and close it or is there something else that needs to happen before tickets can be closed?

Thanks!