I need gigya's set of nested tables (grr!) to be a different width on the log in page, on the comment form, and in a sidebar in a block. Since I don't have the ability to override the inline CSS (see ) and there is only one setting for width on the settings page, I need a way to pass my parameters into the function gigya_loginui_js().

(This may be related to #913984: Allow other modules to modify the LoginUI js but thought I'd post it here anyway.)

CommentFileSizeAuthor
#3 gigya.patch1.43 KBmoonray
#2 login_settings.patch3.51 KBjenlampton

Comments

jenlampton’s picture

jenlampton
she / her
commented

er, I meant see #1030408: rewrite theme system to allow alteration and avoid inline CSS :)

jenlampton’s picture

jenlampton
she / her
commented
Status: Active » Needs review
StatusFileSize
new login_settings.patch3.51 KB

I see the drupal_alter in there, but I think things like width will need to be changed at the theme layer more often than by other modules. This approach leaves both options open.

moonray’s picture

moonray commented
StatusFileSize
new gigya.patch1.43 KB

I have a similar use-case as @jenlampton.

A better way would be to fix the alter to include more context: form_id.
See attached patch.

moonray’s picture

moonray commented
Title: make gigya_loginui_js accept parameters. » Add more context to hook_ gigya_loginui_alter()

Fixing title.

moonray’s picture

moonray commented

Looks #913984: Allow other modules to modify the LoginUI js like has the same solution.

Gigya’s picture

Gigya commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)

Gigya's 6.x module is deprecated. We no longer maintain this version. We encourage upgrading to our latest module.