Page Index Toggle Pages: 1 [2] 3 4  ReplyAdd Poll Send Topic
Very Hot Topic (More than 25 Replies) Re: Looking toward 2.6.1 (Read 30021 times)
 
Paste Member Name in Quick Reply Box Monni
Language
***
Offline


Min izāmō

Posts: 413
Location: Kaarina, Finland
Joined: Jul 16th, 2014
Gender: Male
Mood: Frustrated
Zodiac sign: Pisces
Re: Looking toward 2.6.1
Reply #15 - Aug 25th, 2014 at 8:38pm
Mark & QuoteQuote  
Dandello wrote on Aug 25th, 2014 at 8:16pm:
And this is what I'm seeing in Firefox on my testbed:

Edited:
Now, there may be some minor differences in the default.css file between what's on the test bed and what's in the SVN and I'll double check those in a bit.


I don't have Firefox on this machine anymore... It was a lost cause with constant glitches and need to reinstall... I can debug the current template I have on my laptop though...

Edited:
I just checked the rendering on my laptop and Firefox, and it looks the same as with Chrome and IE. On my client's forum, the alignment is PERFECT. And on this forum it is OK except shifted by two pixels, which is expected.


It shouldn't do that unless the "float: left;" line is missing or on wrong line of the file...

the whole block should read:

Code
Select All
 .ubbcbutton {
     height: 22px;
     width: 23px;
     border: 0;
     margin: 0 1px 1px 0;
     background-position: top right;
     background-repeat: no-repeat;
     text-decoration: none;
     font-size: 18px;
     vertical-align: top;
     display: inline-block;
     float: left;
 }
  

  
Back to top
IP Logged
 
Paste Member Name in Quick Reply Box Dandello
Forum Administrator
YaBB Modder
*****
Offline


I love YaBB 2.7!

Posts: 2234
Location: The Land of YaBB
Joined: Feb 12th, 2014
Gender: Female
Mood: Annoyed
Zodiac sign: Virgo
Re: Looking toward 2.6.1
Reply #16 - Aug 25th, 2014 at 9:21pm
Mark & QuoteQuote  
The float:left was missing - looks good on my testbed. Will upload here in a bit.
  

Perfection is not possible. Excellence, however, is excellent.
Back to top
WWW  
IP Logged
 
Paste Member Name in Quick Reply Box Monni
Language
***
Offline


Min izāmō

Posts: 413
Location: Kaarina, Finland
Joined: Jul 16th, 2014
Gender: Male
Mood: Frustrated
Zodiac sign: Pisces
Re: Looking toward 2.6.1
Reply #17 - Aug 25th, 2014 at 9:22pm
Mark & QuoteQuote  
Dandello wrote on Aug 25th, 2014 at 9:21pm:
The float:left was missing - looks good on my testbed. Will upload here in a bit.


You almost scared me...
  
Back to top
IP Logged
 
Paste Member Name in Quick Reply Box Dandello
Forum Administrator
YaBB Modder
*****
Offline


I love YaBB 2.7!

Posts: 2234
Location: The Land of YaBB
Joined: Feb 12th, 2014
Gender: Female
Mood: Annoyed
Zodiac sign: Virgo
Re: Looking toward 2.6.1
Reply #18 - Aug 25th, 2014 at 10:38pm
Mark & QuoteQuote  
Not having a good day - Took an out-of-town guest out to dinner last night and woke up trying to figure out how to call in sick...  Tongue (I didn't have anything alcoholic so that isn't it.)
  

Perfection is not possible. Excellence, however, is excellent.
Back to top
WWW  
IP Logged
 
Paste Member Name in Quick Reply Box Dandello
Forum Administrator
YaBB Modder
*****
Offline


I love YaBB 2.7!

Posts: 2234
Location: The Land of YaBB
Joined: Feb 12th, 2014
Gender: Female
Mood: Annoyed
Zodiac sign: Virgo
Re: Looking toward 2.6.1
Reply #19 - Aug 26th, 2014 at 3:44am
Mark & QuoteQuote  
Found some other spots where the old css was interferring with the new. Also added a 'use today' option for the Event Calendar.
  

Perfection is not possible. Excellence, however, is excellent.
Back to top
WWW  
IP Logged
 
Paste Member Name in Quick Reply Box Monni
Language
***
Offline


Min izāmō

Posts: 413
Location: Kaarina, Finland
Joined: Jul 16th, 2014
Gender: Male
Mood: Frustrated
Zodiac sign: Pisces
Re: Looking toward 2.6.1
Reply #20 - Aug 26th, 2014 at 7:50am
Mark & QuoteQuote  
lol... getting closer to point where code freeze for 2.6.1 would make sense... Stop fixing bugs and hiccups so we still have enough left to fix for 2.6.2  :Smiley
  
Back to top
IP Logged
 
Paste Member Name in Quick Reply Box Dandello
Forum Administrator
YaBB Modder
*****
Offline


I love YaBB 2.7!

Posts: 2234
Location: The Land of YaBB
Joined: Feb 12th, 2014
Gender: Female
Mood: Annoyed
Zodiac sign: Virgo
Re: Looking toward 2.6.1
Reply #21 - Aug 26th, 2014 at 3:05pm
Mark & QuoteQuote  
Should we fix those bad '||' bits first?  I've spotted some where I can't even figure out what they're suppose to do.

Like line 2782 in Post.pm
Code (Perl)
Select All
            $mname ||= $musername || $post_txt{'470'}; 

  Huh

as near as I can tell it should be
Code (Perl)
Select All
            $mname = $musername || $post_txt{'470'}; 

as $musername should be either 'Guest' or a userID and this is just an error catcher in case it's empty.
  

Perfection is not possible. Excellence, however, is excellent.
Back to top
WWW  
IP Logged
 
Paste Member Name in Quick Reply Box Monni
Language
***
Offline


Min izāmō

Posts: 413
Location: Kaarina, Finland
Joined: Jul 16th, 2014
Gender: Male
Mood: Frustrated
Zodiac sign: Pisces
Re: Looking toward 2.6.1
Reply #22 - Aug 26th, 2014 at 3:13pm
Mark & QuoteQuote  
Dandello wrote on Aug 26th, 2014 at 3:05pm:
Should we fix those bad '||' bits first?  I've spotted some where I can't even figure out what they're suppose to do.

Like line 2782 in Post.pm
Code (Perl)
Select All
            $mname ||= $musername || $post_txt{'470'}; 

  Huh

as near as I can tell it should be
Code (Perl)
Select All
            $mname = $musername || $post_txt{'470'}; 

as $musername should be either 'Guest' or a userID and this is just an error catcher in case it's empty.


||= is supposed to be //= which means set if not defined...

So something like:

Code
Select All
 $mname = defined $musername ? $musername : $post_txt{'470'} unless defined $mname;
  



This is pretty much harakiri code as this doesn't work if $musername or $mname is defined but empty string.
  
Back to top
IP Logged
 
Paste Member Name in Quick Reply Box Dandello
Forum Administrator
YaBB Modder
*****
Offline


I love YaBB 2.7!

Posts: 2234
Location: The Land of YaBB
Joined: Feb 12th, 2014
Gender: Female
Mood: Annoyed
Zodiac sign: Virgo
Re: Looking toward 2.6.1
Reply #23 - Aug 26th, 2014 at 3:24pm
Mark & QuoteQuote  
Code (Perl)
Select All
if ( ! defined $mname || $mname eq q{} ) { $mname = defined $musername ? $musername : $post_txt{'470'} ; }
   

?
  

Perfection is not possible. Excellence, however, is excellent.
Back to top
WWW  
IP Logged
 
Paste Member Name in Quick Reply Box Monni
Language
***
Offline


Min izāmō

Posts: 413
Location: Kaarina, Finland
Joined: Jul 16th, 2014
Gender: Male
Mood: Frustrated
Zodiac sign: Pisces
Re: Looking toward 2.6.1
Reply #24 - Aug 26th, 2014 at 3:28pm
Mark & QuoteQuote  
Dandello wrote on Aug 26th, 2014 at 3:24pm:
Code (Perl)
Select All
if ( ! defined $mname || $mname eq q{} ) { $mname = defined $musername ? $musername : $post_txt{'470'} ; }
   

?


almost... it doesn't check if $musername is empty string yet... But is that even possible?
  
Back to top
IP Logged
 
Paste Member Name in Quick Reply Box Dandello
Forum Administrator
YaBB Modder
*****
Offline


I love YaBB 2.7!

Posts: 2234
Location: The Land of YaBB
Joined: Feb 12th, 2014
Gender: Female
Mood: Annoyed
Zodiac sign: Virgo
Re: Looking toward 2.6.1
Reply #25 - Aug 26th, 2014 at 3:31pm
Mark & QuoteQuote  
Both $mname and $musername come out of a split message string so could actually have a value of empty string - in which case there's more bad things going on with that message string than just not having $mname defined.  Grin

Code (Perl)
Select All
if ( ! defined $mname || $mname eq q{} ) { $mname = (defined $musername && $musername ne q{} ) ? $musername : $post_txt{'470'} ; }
   

  

Perfection is not possible. Excellence, however, is excellent.
Back to top
WWW  
IP Logged
 
Paste Member Name in Quick Reply Box Monni
Language
***
Offline


Min izāmō

Posts: 413
Location: Kaarina, Finland
Joined: Jul 16th, 2014
Gender: Male
Mood: Frustrated
Zodiac sign: Pisces
Re: Looking toward 2.6.1
Reply #26 - Aug 26th, 2014 at 3:33pm
Mark & QuoteQuote  
Dandello wrote on Aug 26th, 2014 at 3:31pm:
Both $mname and $musername come out of a split message string so could actually have a value of empty string - in which case there's more bad things going on with that message string than just not having $mname defined.  Grin

Code (Perl)
Select All
if ( ! defined $mname || $mname eq q{} ) { $mname = (defined $musername || $musername = q{} ) ? $musername : $post_txt{'470'} ; }
   



The bad code usually looks better than the good code... But it's better to make the code safe than fix it twice...
  
Back to top
IP Logged
 
Paste Member Name in Quick Reply Box Dandello
Forum Administrator
YaBB Modder
*****
Offline


I love YaBB 2.7!

Posts: 2234
Location: The Land of YaBB
Joined: Feb 12th, 2014
Gender: Female
Mood: Annoyed
Zodiac sign: Virgo
Re: Looking toward 2.6.1
Reply #27 - Aug 26th, 2014 at 3:35pm
Mark & QuoteQuote  
I was rewriting it as you posted.  Wink
  

Perfection is not possible. Excellence, however, is excellent.
Back to top
WWW  
IP Logged
 
Paste Member Name in Quick Reply Box Monni
Language
***
Offline


Min izāmō

Posts: 413
Location: Kaarina, Finland
Joined: Jul 16th, 2014
Gender: Male
Mood: Frustrated
Zodiac sign: Pisces
Re: Looking toward 2.6.1
Reply #28 - Aug 26th, 2014 at 3:39pm
Mark & QuoteQuote  
Dandello wrote on Aug 26th, 2014 at 3:35pm:
I was rewriting it as you posted.  Wink


It could look a little nicer if we could move the "not defined or empty" test as a sub/function isempty() or something... only parameter to it would be the variable to test and it would return false only if the variable is defined and at least 1 character long.
  
Back to top
IP Logged
 
Paste Member Name in Quick Reply Box Dandello
Forum Administrator
YaBB Modder
*****
Offline


I love YaBB 2.7!

Posts: 2234
Location: The Land of YaBB
Joined: Feb 12th, 2014
Gender: Female
Mood: Annoyed
Zodiac sign: Virgo
Re: Looking toward 2.6.1
Reply #29 - Aug 26th, 2014 at 3:42pm
Mark & QuoteQuote  
According to http://www.perlmonks.org/?node_id=653404 ||= should evaluate as false undefined, empty string and 0 values. Where we may still have some glitches are those places where '0' and empty are a valid values.
  

Perfection is not possible. Excellence, however, is excellent.
Back to top
WWW  
IP Logged
 
Page Index Toggle Pages: 1 [2] 3 4 
ReplyAdd Poll Send Topic
Bookmarks: del.icio.us Digg Facebook Google LinkedIn reddit Twitter Yahoo
Re: Looking toward 2.6.1

Please type the characters exactly as they appear in the image,
without the first 2 and last 2 characters.
The characters must be typed in the same order,
and they are case-sensitive.
Open Preview Preview

You can resize the textbox by dragging the right or bottom border.
Off Topic Comment Insert Spoiler
Insert Hyperlink Insert FTP Link Insert Image Insert E-mail Insert Media Insert Table Insert Table Row Insert Table Column Insert Horizontal Rule Insert Teletype Insert Code Insert Quote Edited Superscript Subscript Insert List /me - my name Insert Marquee Insert Timestamp No Parse
Bold Italicized Underline Insert Strikethrough Highlight
                       
Change Text Color
Insert Preformatted Text Left Align Centered Right Align
resize_wb
resize_hb







Max 5000 characters. Remaining characters:
Text size: %
More Smilies
View All Smilies
Collapse additional features Collapse/Expand additional features Smiley Wink Cheesy Grin Angry Sad Shocked Cool Huh Roll Eyes Tongue Embarrassed Lips Sealed Undecided Kiss Cry