Ticket #145 (closed patch: fixed)

Opened 1 year ago

Last modified 1 year ago

Make video keyword {{video:}} accept more flexible with respect to urls

Reported by: ewout Assigned to: dramirez
Priority: normal Milestone: 0.9
Component: core Version:
Severity: normal Keywords:
Cc: Patch Included:
Review Stage:

Description

Although most people now know where to find the embed code, some people might find it more natural to copy - paste the url in the address bar in the {{video:url}} keyword. This patch takes care of youtube and google video. Maybe other services should be incorporated too? Maybe let's no bother? How about the contenttoolbar popup window, should that also accept more flexible urls? Discuss!

Attachments

new-video-keyword.diff (1.0 kB) - added by ewout on 11/10/07 21:13:54.

Change History

11/10/07 21:13:54 changed by ewout

  • attachment new-video-keyword.diff added.

11/11/07 17:19:10 changed by dramirez

  • status changed from new to closed.
  • resolution set to fixed.

11/11/07 20:14:14 changed by ewout

Diego, you closed my ticket without comment. Don't you like it? Is something wrong with it? Or did you apply it without checking in? Please discuss.

(follow-up: ↓ 5 ) 11/12/07 02:47:04 changed by dramirez

  • status changed from closed to reopened.
  • resolution deleted.

Sorry Ewout. Wrong option. The idea was assign me the ticket no close it. I guess I do the same with other tickets :-/.

I will check your patch and comment after that.

11/12/07 02:47:19 changed by dramirez

  • owner changed from misja to dramirez.
  • status changed from reopened to new.
  • milestone set to 0.9.

(in reply to: ↑ 3 ) 11/12/07 09:18:15 changed by ewout

Replying to dramirez:

Sorry Ewout. Wrong option. The idea was assign me the ticket no close it. I guess I do the same with other tickets :-/.

No problem :-)

12/03/07 22:09:44 changed by misja

  • status changed from new to closed.
  • resolution set to fixed.

applied in r1334