Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Comment: Migrated to Confluence 4.0

...

[10:53:52 CDT(-0500)] <cindyli> this path can be relative or absolute. but since you are accessing template thru a web server, absolute paths may not suitable or work, but you can certainly use the web absolute url such as "http://mysite/UIO-template-path&#034; " as long as your template directory is exposed to the public

...

[12:12:38 CDT(-0500)] <anastasiac> thanks for the link to the enactor tests. I'm not sure that's enough for what I'm doing. In IE8, we don't support captions and we don't want the video player to add the captions settings panel. When the test file runs in non-IE8, the test needs to verify that the panel is there, but when it runs in IE8, the test needs to verify the opposite. I know I can use if/else, but I'm wondering if there's something cleaner

[12:18:46 CDT(-0500)] <cindyli1> anastasiac: i wonder if you can use demands block to register different test function based on the video support, such as https://github.com/fluid-project/videoPlayer/blob/master/js/VideoPlayer.js#L303

[12:19:17 CDT(-0500)] <anastasiac> cindyli, I tried that, it didn't seem to work for test functions. the functions were not resolved based on the blocks

[12:20:21 CDT(-0500)] <cindyli1> ah, interesting

[12:27:33 CDT(-0500)] <cindyli1> anastasiac: aother way is to write a function that takes in two arguments which are test functions for non-IE8 and IE8 enviroment. this function checks if "fluid.browser.nativeVideoSupport" is in the static environment and run proper test function accordingly

[12:28:27 CDT(-0500)] <anastasiac> cindyli1, that sounds not bad. cleaner than if/then right in a single test function. Thanks for the suggestion

[12:28:37 CDT(-0500)] <cindyli1> np

[13:01:54 CDT(-0500)] <anastasiac> http://piratepad.net/JSiISW8sp0

[13:22:44 CDT(-0500)] <anastasiac> schema pirate pad: http://piratepad.net/U8WovnfY6n

[13:58:20 CDT(-0500)] <cindyli1> hi yzen

[13:58:44 CDT(-0500)] <yzen> cindyli1: hi

[13:59:21 CDT(-0500)] <cindyli1> yzen: in terms of the UIO localization code - https://github.com/fluid-project/infusion/blob/master/src/webapp/components/uiOptions/js/i18n.js#L63-65

[14:00:16 CDT(-0500)] <yzen> cindyli1: yes

[14:00:18 CDT(-0500)] <cindyli1> after i18n.js is loaded, can we overwrite the strings defined in "fluid.uiOptions.messages" before instantiate UIO and expect the strings displayed in the panel get changed?

[14:00:48 CDT(-0500)] <cindyli1> such as:

[14:00:58 CDT(-0500)] <cindyli1> fluid.uiOptions.messages.textFontLabel = "new font";

[14:00:58 CDT(-0500)] <cindyli1> fluid.uiOptions.fatPanel(container, commonOpts);

[14:02:25 CDT(-0500)] <yzen> i believe so, it's not frozen

[14:04:06 CDT(-0500)] <cindyli1> no, but the string is not updated with the new one, yzen

[14:05:26 CDT(-0500)] <yzen> cindyli1: right, makes sense, options for it are copied, we can avoid it if we "preserve" the messageBase option with mergePolicy

[14:06:06 CDT(-0500)] <yzen> so we can have a grade: fluid.uiOptions.messageResolver that is a "fluid.messageResolver" along with a custom merge policy

[14:06:17 CDT(-0500)] <yzen> this will let you modify the base from the outside

[14:11:57 CDT(-0500)] <cindyli1> yzen, Justin_o and I were thinking what's the best way for integrators to modify those strings. It doesn't make sense to modify i18n.js. If we provide an API, the auxiliary schema might be the best place for it. What do you feel about it?

[14:12:26 CDT(-0500)] <yzen> cindyli1: i think that would be best

[14:14:01 CDT(-0500)] <cindyli1> ok, thanks, yzen

[14:14:26 CDT(-0500)] <Justin_o> cindyli1: would you want to take in a message bundle into the schema to be merged with anything from the i18n.js

[14:14:27 CDT(-0500)] <Justin_o> ?

[14:15:01 CDT(-0500)] <cindyli1> yes, Justin_o

[14:33:42 CDT(-0500)] <cindyli1> anastasiac: Bosmon, yzen, the updated auxiliary schema - http://wiki.fluidproject.org/display/fluid/Auxiliary+Schema+for+UIO+Preferences, your feedback please

[14:38:42 CDT(-0500)] <anastasiac> thanks, cindyli1