bootstrap-wysihtml5 icon indicating copy to clipboard operation
bootstrap-wysihtml5 copied to clipboard

Inserting YouTube videos

Open elzos opened this issue 12 years ago • 11 comments

  • My first attempt, fixed width and height to minimal.
  • Works with typical youtube URLs like 'http://www.youtube.com/watch?v=...'.
  • Updates in en, el locales (video key). Needed for other locales.
  • No changes made to underlying editor.
  • Added functions in basic js file at the bottom. They can be transferred to separate file(s).
  • Rake to dist pending... please review it works as expected first!

elzos avatar Nov 12 '12 20:11 elzos

I tried this commit and it works perfectly! Thanks.

SRWieZ avatar Feb 16 '13 11:02 SRWieZ

Thats for spanish !

  •        video: {
    
  •            insert: "Insertar Video de YouTube",
    
  •            cancel: "Cancelar",
    
  •            invalid: "URL no valida"
    
  •        },
    

that will be pulled!! 100% Recommended!

SpaHost avatar Apr 01 '13 13:04 SpaHost

+1

hyperrjas avatar May 16 '13 18:05 hyperrjas

+1

tomkrus avatar May 23 '13 10:05 tomkrus

+1 was an simple merge with current head (commit: 283bb8f859977ac1aafb09bee715a1190f8cfd1b)

As seen in https://github.com/puredistortion/bootstrap-wysihtml5/commit/7e2c149dd2a3b6f416dae97d79c4d585eae7a7e9

dalethestirling avatar May 26 '13 23:05 dalethestirling

@puredistortion I'm using https://github.com/Nerian/bootstrap-wysihtml5-rails and I can't use this commit with this gem.

Thank you!

hyperrjas avatar Jun 19 '13 17:06 hyperrjas

+1

ericcbm avatar Aug 02 '13 12:08 ericcbm

@hyperrjas I have merged the javascript see Pull Request though I had no idea how to get Rails working so it will need some testing.

dalethestirling avatar Aug 13 '13 12:08 dalethestirling

Hi,

There is a TestApp you can use to test the gem https://github.com/Nerian/bootstrap-wysihtml5-rails/tree/master/testapp. I tested it and it works well :thumbsup:

I will merge it as soon it is merged in this repo.

Nerian avatar Aug 13 '13 14:08 Nerian

+1 on this PR. Please merge this :)

AlJohri avatar Aug 25 '13 23:08 AlJohri

Translation does not work, need to update too.

tomkrus avatar Aug 26 '13 05:08 tomkrus