I think its bad that the size is in px, so its hard to change it for custom themes.

There are 2 possibilites:

  1. Write a Js Theme Function
  2. Change height to "auto", which worked here like a charm

I would write a patch, if I know which one should be done.

Comments

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented
Category: bug » feature

I'm afraid we're constrained by jQuery UI Dialog, where the height option should be specified in pixels.

http://jqueryui.com/demos/dialog/

However, I could change the code to get the dialog dimensions from a function that can be overridden by themers. Something like:

Drupal.theme.prototype.CCKPrivateFieldsDialogSize = function() {
  return {width: 500, height: 250};
};

Will do as soon as I have the time.

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented
Title: Popup size is in px » Allow themers override the modal dialog size (in pixels)

Better title. Feature will be added to CVS shortly...

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented
Status: Active » Fixed

Committed to CVS (patch).

It should be available in the next development snapshot.

If no other issues arise, I'll release a new beta in a couple of days.

Thanks for the suggestion.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.