GCPRO1(descriptor_item);
/* Pick up any generic properties that we might need to keep hold
- of.
+ of.
#### This is potentially bogus because it is changing the items
in place rather than in the pending items. */
if (!NILP(text)) {
keywords.
#### This is inconsistent with instantiation in that you have to
- have the :descriptor keyword for updates in order to recognise
+ have the :descriptor keyword for updates in order to recognise
changes. */
if (VECTORP(desc)) {
descriptor_item = gui_parse_item_keywords_no_errors(desc);
keywords. Note that standard gui descriptor shortcuts will not work
because of keyword parsing.
- #### This is bogus in that descriptor and items share the same slot,
+ #### This is bogus in that descriptor and items share the same slot,
we should rationalize. */
if (VECTORP(desc)) {
IMAGE_INSTANCE_WIDGET_ITEMS(ii) =
glyphs are cached on a device basis like most other glyphs. Instead
they should be cached per-window and then the instance would be
fixed and we wouldn't have to mess around with font metrics and the
- rest.
+ rest.
Another sizing problem is alignment. We provide layout widgets that
allow users to stack widgets vertically or horizontally. These
/* Work out minimum space we need to fit all the items. This could
have been fixed by the user. */
- if (IMAGE_INSTANCE_SUBWINDOW_H_RESIZEP(ii)) {
+ if (width && IMAGE_INSTANCE_SUBWINDOW_H_RESIZEP(ii)) {
if (!NILP(IMAGE_INSTANCE_WIDGET_WIDTH_SUBR(ii))) {
Lisp_Object dynamic_width =
Feval(IMAGE_INSTANCE_WIDGET_WIDTH_SUBR(ii));
}
/* Work out vertical spacings. */
- if (IMAGE_INSTANCE_SUBWINDOW_V_RESIZEP(ii)) {
+ if (height && IMAGE_INSTANCE_SUBWINDOW_V_RESIZEP(ii)) {
if (!NILP(IMAGE_INSTANCE_WIDGET_HEIGHT_SUBR(ii))) {
Lisp_Object dynamic_height =
Feval(IMAGE_INSTANCE_WIDGET_HEIGHT_SUBR(ii));