extern "C" void C_Combox_combo_cb(FL_OBJECT *ob, long data) ;
extern "C" int C_Combox_peek_event(FL_FORM * form, void *xev);
-Combox::Combox(combox_type t): type(t)
+Combox::Combox(combox_type t)
+ : type(t), tabfolder1(0), tabfolder2(0)
{
browser = button = 0;
callback = 0;
}
-void Combox::add(int x, int y, int w, int hmin, int hmax)
-{
+void Combox::add(int x, int y, int w, int hmin, int hmax,
+ FL_OBJECT * tabfolder1_, FL_OBJECT * tabfolder2_)
+{
+ // Store these for later use in working round an xforms bug in Show()
+ tabfolder1 = tabfolder1_;
+ tabfolder2 = tabfolder2_;
+
FL_OBJECT * obj;
- switch(type) {
+ switch (type) {
case FL_COMBOX_DROPLIST:
{
button = obj = fl_add_button(FL_NORMAL_BUTTON,
if (_pre) _pre();
int tmp;
- XGetInputFocus(fl_display, &save_window, &tmp); //BUG-Fix Dietmar
- XFlush(fl_display);
+ XGetInputFocus(fl_get_display(), &save_window, &tmp); //BUG-Fix Dietmar
+ XFlush(fl_get_display());
if (button && type != FL_COMBOX_NORMAL) {
fl_set_object_label(button, "@2<-");
fl_redraw_object(button);
}
- int x = label->form->x + label->x, y = label->form->y + label->y;
- fl_set_form_position(form, x, y + label->h);
+
+ int x = label->x;
+ int y = label->y + label->h;
+ if (tabfolder1) {
+ // This is a bug work around suggested by Steve Lamont on the
+ // xforms mailing list. It correctly positions the browser form
+ // after the main window has been moved.
+ // The bug only occurs in tabbed folders.
+ int folder_x, folder_y, folder_w, folder_h;
+ fl_get_folder_area( tabfolder1,
+ &folder_x, &folder_y,
+ &folder_w, &folder_h );
+ x += folder_x;
+ y += folder_y;
+
+ if (tabfolder2) {
+ fl_get_folder_area( tabfolder2,
+ &folder_x, &folder_y,
+ &folder_w, &folder_h );
+ x += tabfolder2->form->x + folder_x;
+ y += tabfolder2->form->y + folder_y;
+ } else {
+ x += tabfolder1->form->x;
+ y += tabfolder1->form->y;
+ }
+
+ } else {
+ x += label->form->x;
+ y += label->form->y;
+ }
+
+ fl_set_form_position(form, x, y);
fl_show_form(form, FL_PLACE_POSITION, FL_NOBORDER, "");
if (sel>0) {
fl_set_browser_topline(browser, sel);
fl_select_browser_line(browser, sel);
}
- XGrabPointer(fl_display, form->window, false,
+ XGrabPointer(fl_get_display(), form->window, false,
ButtonPressMask | ButtonReleaseMask | PointerMotionMask,
GrabModeAsync, GrabModeAsync,
0, 0, 0);
- XFlush(fl_display);
+ XFlush(fl_get_display());
}
void Combox::Hide(int who)
fl_get_browser_line(browser, sel));
// if (callback) callback(sel, cb_arg);
}
- XUngrabPointer(fl_display, 0);
- XFlush(fl_display);
+ XUngrabPointer(fl_get_display(), 0);
+ XFlush(fl_get_display());
if (!who && browser && label && callback)
callback(sel, cb_arg, this);
if (form) {
fl_hide_form(form);
- XSetInputFocus(fl_display, save_window,
+ XSetInputFocus(fl_get_display(), save_window,
RevertToParent, CurrentTime); // BUG-FIX-Dietmar
- XFlush(fl_display);
+ XFlush(fl_get_display());
}
if (button) {
if (type != FL_COMBOX_NORMAL){
if (label) fl_deactivate_object(label);
}
+
void Combox::input_cb(FL_OBJECT * ob, long)
{
Combox * combo = static_cast<Combox*>(ob->u_vdata);
char const * text = fl_get_input(ob);
- combo->addto(text);
+ combo->addto(text ? text : string());
combo->is_empty = false;
}
+
extern "C" void C_Combox_input_cb(FL_OBJECT * ob, long data)
{
Combox::input_cb(ob, data);
switch (data) {
case 0:
{
- int i = combo->get();
+ int const i = combo->get();
switch (fl_get_button_numb(ob)) {
case 2:
{
- combo->select(--i);
+ combo->select(i - 1);
if (combo->callback)
combo->callback(combo->sel,
combo->cb_arg, combo);
}
case 3:
{
- combo->select(++i);
+ combo->select(i + 1);
if (combo->callback)
combo->callback(combo->sel,
combo->cb_arg, combo);
if (static_cast<XEvent*>(xev)->type != KeyPress) return 0;
char s_r[10]; s_r[9] = '\0';
- static int num_bytes;
KeySym keysym_return;
- num_bytes = XLookupString(&static_cast<XEvent*>(xev)->xkey, s_r, 10,
- &keysym_return, 0);
- XFlush(fl_display);
+ XLookupString(&static_cast<XEvent*>(xev)->xkey, s_r, 10,
+ &keysym_return, 0);
+ XFlush(fl_get_display());
switch (keysym_return) {
case XK_Down:
if (fl_get_browser(combo->browser) <