maxmustermann/godot
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Tabs: cleanup code, no functional change

Browse source 
Some dead code removed, a few cosmetic changes.
This commit is contained in:
Rémi Verschelde 2016-05-01 16:27:33 +02:00
parent 8574489dd2
commit 8b32c80529
1 changed files with 36 additions and 86 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

122
scene/gui/tabs.cpp
View file

@ -32,25 +32,23 @@
Size2 Tabs::get_minimum_size() const {
Ref<StyleBox> tab_bg = get_stylebox("tab_bg");
Ref<StyleBox> tab_fg = get_stylebox("tab_fg");
Ref<Font> font = get_font("font");
Size2 ms(0, MAX( tab_bg->get_minimum_size().height,tab_fg->get_minimum_size().height)+font->get_height() );
// h+=MIN( get_constant("label_valign_fg"), get_constant("label_valign_bg") );
Size2 ms(0, MAX(tab_bg->get_minimum_size().height, tab_fg->get_minimum_size().height)+font->get_height());
for(int i=0;i<tabs.size();i++) {
Ref<Texture> tex = tabs[i].icon;
if (tex.is_valid()) {
ms.height = MAX( ms.height, tex->get_size().height );
ms.height = MAX(ms.height, tex->get_size().height);
if (tabs[i].text!="")
ms.width+=get_constant("hseparation");
}
ms.width+=font->get_string_size(tabs[i].text).width;
if (current==i)
ms.width+=tab_fg->get_minimum_size().width;
else
@ -58,28 +56,26 @@ Size2 Tabs::get_minimum_size() const {
if (tabs[i].right_button.is_valid()) {
Ref<Texture> rb=tabs[i].right_button;
Size2 bms = rb->get_size();//+get_stylebox("button")->get_minimum_size();
Size2 bms = rb->get_size();
bms.width+=get_constant("hseparation");
ms.width+=bms.width;
ms.height=MAX(bms.height+tab_bg->get_minimum_size().height,ms.height);
}
if (cb_displaypolicy==CLOSE_BUTTON_SHOW_ALWAYS || (cb_displaypolicy==CLOSE_BUTTON_SHOW_ACTIVE_ONLY && i==current)) {
Ref<Texture> cb=get_icon("close");
Size2 bms = cb->get_size();//+get_stylebox("button")->get_minimum_size();
Size2 bms = cb->get_size();
bms.width+=get_constant("hseparation");
ms.width+=bms.width;
ms.height=MAX(bms.height+tab_bg->get_minimum_size().height,ms.height);
}
}
ms.width=0; //should make this optional
ms.width=0; //TODO: should make this optional
return ms;
}
void Tabs::_input_event(const InputEvent& p_event) {
if (p_event.type==InputEvent::MOUSE_MOTION) {
@ -101,15 +97,14 @@ void Tabs::_input_event(const InputEvent& p_event) {
}
}
// test hovering to display right or close button
int hover_buttons=-1;
hover=-1;
for(int i=0;i<tabs.size();i++) {
// test hovering tab to display close button if policy says so
if (i<offset)
continue;
// test hovering right button and close button
if (tabs[i].rb_rect.has_point(pos)) {
rb_hover=i;
cb_hover=-1;
@ -123,11 +118,9 @@ void Tabs::_input_event(const InputEvent& p_event) {
break;
}
}
if (hover_buttons == -1) { // no hover
if (hover_buttons == -1) { // no hover
rb_hover= hover_buttons;
cb_hover= hover_buttons;
}
@ -137,8 +130,6 @@ void Tabs::_input_event(const InputEvent& p_event) {
}
if (rb_pressing && p_event.type==InputEvent::MOUSE_BUTTON &&
!p_event.mouse_button.pressed &&
p_event.mouse_button.button_index==BUTTON_LEFT) {
@ -152,9 +143,10 @@ void Tabs::_input_event(const InputEvent& p_event) {
update();
}
if (cb_pressing && p_event.type==InputEvent::MOUSE_BUTTON &&
!p_event.mouse_button.pressed &&
p_event.mouse_button.button_index==BUTTON_LEFT) {
!p_event.mouse_button.pressed &&
p_event.mouse_button.button_index==BUTTON_LEFT) {
if (cb_hover!=-1) {
//pressed
@ -195,12 +187,12 @@ void Tabs::_input_event(const InputEvent& p_event) {
}
}
int found=-1;
for(int i=0;i<tabs.size();i++) {
if (i<offset)
continue;
if (tabs[i].rb_rect.has_point(pos)) {
rb_pressing=true;
update();
@ -213,10 +205,7 @@ void Tabs::_input_event(const InputEvent& p_event) {
return;
}
int ofs=tabs[i].ofs_cache;
int size = tabs[i].ofs_cache;
if (pos.x >=tabs[i].ofs_cache && pos.x<tabs[i].ofs_cache+tabs[i].size_cache) {
found=i;
break;
}
@ -232,8 +221,8 @@ void Tabs::_input_event(const InputEvent& p_event) {
}
void Tabs::_notification(int p_what) {
void Tabs::_notification(int p_what) {
switch(p_what) {
@ -259,39 +248,20 @@ void Tabs::_notification(int p_what) {
Ref<Texture> close=get_icon("close");
int h = get_size().height;
int label_valign_fg = get_constant("label_valign_fg");
int label_valign_bg = get_constant("label_valign_bg");
int w=0;
int w = 0;
int mw = 0;
{
// h+=MIN( get_constant("label_valign_fg"), get_constant("label_valign_bg") );
for(int i=0;i<tabs.size();i++) {
int sz = get_tab_width(i);
tabs[i].ofs_cache=mw;
mw+=sz;
}
for(int i=0;i<tabs.size();i++) {
tabs[i].ofs_cache = mw;
mw += get_tab_width(i);
}
if (tab_align==ALIGN_CENTER) {
w=(get_size().width-mw)/2;
} else if (tab_align==ALIGN_RIGHT) {
w=get_size().width-mw;
}
if (w<0) {
@ -311,6 +281,7 @@ void Tabs::_notification(int p_what) {
if (i<offset)
continue;
tabs[i].ofs_cache=w;
int lsize = get_tab_width(i);
@ -327,18 +298,14 @@ void Tabs::_notification(int p_what) {
}
Ref<StyleBox> sb;
int va;
Color col;
if (i==current) {
sb=tab_fg;
va=label_valign_fg;
col=color_fg;
} else {
sb=tab_bg;
va=label_valign_bg;
col=color_bg;
}
@ -363,6 +330,7 @@ void Tabs::_notification(int p_what) {
w+=slen;
if (tabs[i].right_button.is_valid()) {
Ref<StyleBox> style = get_stylebox("button");
Ref<Texture> rb=tabs[i].right_button;
@ -387,10 +355,8 @@ void Tabs::_notification(int p_what) {
w+=style->get_margin(MARGIN_RIGHT);
tabs[i].rb_rect=rb_rect;
}
if (cb_displaypolicy==CLOSE_BUTTON_SHOW_ALWAYS || (cb_displaypolicy==CLOSE_BUTTON_SHOW_ACTIVE_ONLY && i==current)) {
Ref<StyleBox> style = get_stylebox("button");
@ -410,11 +376,8 @@ void Tabs::_notification(int p_what) {
style->draw(ci,cb_rect);
}
//w+=style->get_margin(MARGIN_LEFT);
cb->draw(ci,Point2i( w,cb_rect.pos.y+style->get_margin(MARGIN_TOP) ));
w+=cb->get_width();
//w+=style->get_margin(MARGIN_RIGHT);
tabs[i].cb_rect=cb_rect;
}
@ -429,28 +392,26 @@ void Tabs::_notification(int p_what) {
int vofs = (get_size().height-incr->get_size().height)/2;
if (offset>0)
draw_texture(hilite_arrow==0?decr_hl:decr,Point2(limit,vofs));
draw_texture(hilite_arrow==0 ? decr_hl : decr, Point2(limit,vofs));
else
draw_texture(decr,Point2(limit,vofs),Color(1,1,1,0.5));
draw_texture(decr,Point2(limit,vofs), Color(1,1,1,0.5));
if (missing_right)
draw_texture(hilite_arrow==1?incr_hl:incr,Point2(limit+decr->get_size().width,vofs));
draw_texture(hilite_arrow==1 ? incr_hl : incr, Point2(limit+decr->get_size().width,vofs));
else
draw_texture(incr,Point2(limit+decr->get_size().width,vofs),Color(1,1,1,0.5));
draw_texture(incr,Point2(limit+decr->get_size().width,vofs), Color(1,1,1,0.5));
buttons_visible=true;
} else {
buttons_visible=false;
}
} break;
}
}
int Tabs::get_tab_count() const {
return tabs.size();
}
@ -459,11 +420,9 @@ void Tabs::set_current_tab(int p_current) {
ERR_FAIL_INDEX( p_current, get_tab_count() );
//printf("DEBUG %p: set_current_tab to %i\n", this, p_current);
current=p_current;
_change_notify("current_tab");
//emit_signal("tab_changed",current);
update();
}
@ -487,9 +446,9 @@ String Tabs::get_tab_title(int p_tab) const{
ERR_FAIL_INDEX_V(p_tab,tabs.size(),"");
return tabs[p_tab].text;
}
void Tabs::set_tab_icon(int p_tab,const Ref<Texture>& p_icon){
ERR_FAIL_INDEX(p_tab,tabs.size());
@ -498,6 +457,7 @@ void Tabs::set_tab_icon(int p_tab,const Ref<Texture>& p_icon){
minimum_size_changed();
}
Ref<Texture> Tabs::get_tab_icon(int p_tab) const{
ERR_FAIL_INDEX_V(p_tab,tabs.size(),Ref<Texture>());
@ -506,7 +466,6 @@ Ref<Texture> Tabs::get_tab_icon(int p_tab) const{
}
void Tabs::set_tab_right_button(int p_tab,const Ref<Texture>& p_right_button){
ERR_FAIL_INDEX(p_tab,tabs.size());
@ -556,8 +515,6 @@ void Tabs::remove_tab(int p_idx) {
if (current>=tabs.size())
current=tabs.size()-1;
//emit_signal("tab_changed",current);
_ensure_no_over_offset();
}
@ -581,7 +538,7 @@ int Tabs::get_tab_width(int p_idx) const {
Ref<StyleBox> tab_bg = get_stylebox("tab_bg");
Ref<StyleBox> tab_fg = get_stylebox("tab_fg");
Ref<Font> font = get_font("font");
Ref<Texture> close=get_icon("close");
int x=0;
@ -593,8 +550,8 @@ int Tabs::get_tab_width(int p_idx) const {
}
x+=font->get_string_size(tabs[p_idx].text).width;
if (current==p_idx)
x+=tab_fg->get_minimum_size().width;
else
@ -602,17 +559,14 @@ int Tabs::get_tab_width(int p_idx) const {
if (tabs[p_idx].right_button.is_valid()) {
Ref<Texture> rb=tabs[p_idx].right_button;
Size2 bms = rb->get_size();//+get_stylebox("button")->get_minimum_size();
bms.width+=get_constant("hseparation");
x+=bms.width;
x+=rb->get_width();
x+=get_constant("hseparation");
}
if (cb_displaypolicy==CLOSE_BUTTON_SHOW_ALWAYS || (cb_displaypolicy==CLOSE_BUTTON_SHOW_ACTIVE_ONLY && p_idx==current)) {
Size2 bms = close->get_size();//+get_stylebox("button")->get_minimum_size();
bms.width+=get_constant("hseparation");
x+=bms.width;
Ref<Texture> cb=get_icon("close");
x+=cb->get_width();
x+=get_constant("hseparation");
}
return x;
@ -668,11 +622,9 @@ void Tabs::ensure_tab_visible(int p_idx) {
Ref<Texture> incr = get_icon("increment");
Ref<Texture> decr = get_icon("decrement");
int limit=get_size().width-incr->get_width()-decr->get_width();
int x=0;
for(int i=0;i<tabs.size();i++) {
@ -718,7 +670,6 @@ void Tabs::_bind_methods() {
ADD_SIGNAL(MethodInfo("right_button_pressed",PropertyInfo(Variant::INT,"tab")));
ADD_SIGNAL(MethodInfo("tab_close",PropertyInfo(Variant::INT,"tab")));
ADD_PROPERTY( PropertyInfo(Variant::INT, "current_tab", PROPERTY_HINT_RANGE,"-1,4096,1",PROPERTY_USAGE_EDITOR), _SCS("set_current_tab"), _SCS("get_current_tab") );
BIND_CONSTANT( ALIGN_LEFT );
@ -742,9 +693,8 @@ Tabs::Tabs() {
cb_hover=-1;
cb_pressing=false;
cb_displaypolicy = CLOSE_BUTTON_SHOW_NEVER; // Default : no close button
cb_displaypolicy = CLOSE_BUTTON_SHOW_NEVER;
offset=0;
max_drawn_tab=0;
}