Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#12273 closed enhancement (fixed)

[Interface Kit] alert windows are too wide

Reported by: diver Owned by: axeld
Priority: high Milestone: R1/beta1
Component: Kits/Interface Kit Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description (last modified by diver)


It seems that when in hrev49481 BAlert made its transition to layout API it has also changed the way alert windows look. Would it be possible to make them tidy again?

Old look: New look:

Note that Sceenshot doesn't capture window borders of alert windows -- #11368.

Attachments (2)

alert_old.png (9.5 KB ) - added by diver 5 years ago.
alert_new.png (11.8 KB ) - added by diver 5 years ago.

Download all attachments as: .zip

Change History (9)

by diver, 5 years ago

Attachment: alert_old.png added

by diver, 5 years ago

Attachment: alert_new.png added

comment:1 by diver, 5 years ago

Description: modified (diff)

comment:2 by waddlesplash, 5 years ago

Milestone: UnscheduledR1/beta1
Priority: normalhigh

comment:3 by humdinger, 5 years ago

Is this really a high prio beta-blocker?

comment:4 by waddlesplash, 5 years ago

Resolution: fixed
Status: newclosed

Fixed in hrev49505.

comment:5 by axeld, 5 years ago

I don't agree with this, actually. This is why I made it the way it was. Why should it use different spacing than what is used everywhere else?

Alerts are very small anyway, so there is no reason to make them that crowded. I'd like to see that reverted or at least come up with an argument why it should use that spacing other than "that is how it used to look".

comment:6 by jua, 5 years ago

It would be nice though to give the buttons an identical width again, which looks nicer IMO. Having them generally a bit wider is not bad as well (like in the old version), it makes them easier click targets.

comment:7 by pulkomandy, 5 years ago

Keeping the text top-aligned with the icon as in the "old" look looks better to me. I don't mind some extra spacing, however.

Note: See TracTickets for help on using tickets.