Closed Bug 606331 Opened 14 years ago Closed 14 years ago

Link timestamps to tweet permalink

Categories

(support.mozilla.org :: Army of Awesome, task)

task
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: williamr, Assigned: wenzel)

References

()

Details

Link tweet's timestamp so it will open a new tab/window to that tweet's permalink.

Example permalink: http://twitter.com/#!/dailycavalier/status/26406528177

Link will change color to a blue when in a hover state.
Assignee: malexis → nobody
Target Milestone: --- → 2.2.7
Assignee: nobody → malexis
Fred, this is part of customer care v1.1

Target launch date is Nov 3
Assignee: malexis → fwenzel
More goodness:

http://github.com/fwenzel/kitsune/commit/c319199

For extra credit, I keep clicks on links from bubbling up the DOM tree now, so our signin popup doesn't come up when you click a link.

jsocol: r? please!
Status: NEW → ASSIGNED
Nice job with extra credit :)

Not sure if this matters, but I noticed that the new twitter.com is now using a slightly different permalink structure (adds "#!/" before username). However the old permalinks redirect properly to the new structure.

Old permalink: http://twitter.com/{{ tweet.user }}/status/{{ tweet.id }}
New permalink: http://twitter.com/#!/{{ tweet.user }}/status/{{ tweet.id }}

Should we be using the new permalink structure? Or is that still experimental?
When I go to twitter without being logged in, I get "old-style" links, so I think we are fine using those.
Great, that sounds good to me.
Landed on 2.2.x and master.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Verified timestamps link correctly to tweet
Status: RESOLVED → VERIFIED
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.