Webdialogs: inputbox shows black background (sketchup webkit

Juantxo

When I updated to Safari to 5.1.7 in Mac Leopard webdialogs boxes turn black in Sketchup.
I've installed an older version of safari with pacifist and now it works again.

http://www.charlessoft.com/ (pacifist software)
http://mac.oldapps.com/safari.php (Safari 5.1.1 (Snow Leopard))

jeff hammond

@lwerner said:

Thanks for posting this workaround! It worked perfectly for me. (On OSX 10.7.4 Lion, Sketchup 8.0.14345 Pro)

Laura

ditto.. works fine for me too..
thanks peter.

TeamSDA

Hi All,

The fix Peter provided worked for the fields in my DC options window. But when I open my custom attributes window and look at the "Display Label" it is black as well. I couldn't find the style in the css style sheet that changed this input field. Does anyone know the style I need to change.

Custom Attributes

(On OSX 10.6.8 Snow Leopard, Sketchup 8.0.14345 Pro)

Regards,
Christian

mwm5053

@juantxo said:

When I updated to Safari to 5.1.7 in Mac Leopard webdialogs boxes turn black in Sketchup.
I've installed an older version of safari with pacifist and now it works again.

http://www.charlessoft.com/ (pacifist software)
http://mac.oldapps.com/safari.php (Safari 5.1.1 (Snow Leopard))

Update If you find more black dialoge boxes blacked out like in profile builder or at 3d warehose this solution of Juantxo will work for all blacked
out dialoge boxes follow instructions on this page to revert Safari back to 5.1.1 even though it's for a earlier ver.

(appletoolbox.com)

jeff hammond

fwiw, i did the DC fix a while ago and yes, it works but as mentioned in this thread, it only works with the DC dialog.. skindigo was also exhibiting this blacked out behavior so i eventually downgraded safari using the method mentioned by Juantxo a few posts back.. pacifist with an older version of safari..

and that fixes everything in sketchup instead of just the DC dialog.

mwm5053

I wonder if this has been reported to apple so the next Safari update will resolve this? BTW Still love my MAC

driven

I'm using the developer release of Safari 6 and the problem remains in SU.

Safari's User Agent Style Sheet, handles the input border correctly but SU doesn't use that, so it's a WebCore update that's needed. i'm looking for a patch at the moment.

john

mwm5053

Thanks John I'd like to stay updated on this as I like to stay updated on all software.

gohch

Thanks guys!
I managed to get all the dialog boxes of 1001bit fixed following the suggestions on the css;

patch in html file to fix the black textbox

It's a lot of work to edit all the html files, but at least the problem is solved.
Cheers!

RichMorin

I grabbed a copy of Pacifist, but I can't find a copy of Safari 5.1.5 to install.

For example, the page http://support.apple.com/kb/DL1070
has a Download button that leads to http://www.apple.com/safari/download/.

However, this page only supports downloading of Safari 5.1.7.

Help?

troypiggo

I found it here:

http://mac.oldapps.com/safari.php?old_safari=813

driven

Hi,
update to my investigation.

WebView uses a different version of Webkit than Safari (and not just in SU) and I can't find a way to change the userAgent from within SU, so been trying another approach.

dlg.execute_script %(function addGlobalStyle(css) { 
   var head, style; 
   head = document.getElementsByTagName('head')[0]; 
   if (!head) { return; } 
   style = document.createElement('style'); 
   style.type = 'text/css'; 
   style.innerHTML = css; 
   head.appendChild(style); 

} 

addGlobalStyle('input {border; 1px solid gray;}'))

This works if you know the name of the dialog, but how can I get that with ruby to apply a global patch. Is it possible to get a new dialog name in ruby.
Using WebInspector to add the JS I can also exclude type with

addGlobalStyle('input, input;not[type="submit"];not[type="button"];not[type="file"];not[type="search"]{border; 1px solid gray;}'))

, but that returns true and fails in RubyConsole
john

thomthom

@aerilius said:

UI::WebDialog.open

You mean UI::WebDialog.show ?

Aerilius

fixed it.

driven

@aerilius said:

fixed it.

hi, I'm getting
Error: #<SyntaxError: (eval):24: compile error (eval):15: parse error, unexpected '(' alias(:show_orig, :show) ^ (eval):20: both block arg and actual block given> (eval):24

I'd much rather use a small amount of ruby to 'fix' the input problem

cheers for your efforts
john

Dan Rathbun

@Aerilius

It's much more complicated than that.

I have a patch that does similar.. (and more,) but we can't release such a thing without Boulder's blessing.

We have realized, that it's best not to show how to modify the API modules and classes. Some newbie's just run with it and start hacking away, and worse. (Yes we've all been guilty of openly discussing such things in the past. I'm sure I have. Sometimes we forget the "kids" come along and read what we've written later.)

Anyway... you don't need the literal block when you pass to the orig method, as the wrapper method has already converted any block argument to a Proc object.
So instead you do:
block_given? ? show_orig(&proc) : show_orig()

But it will not work like that, for the same reasons that the current API show(), show_modal() and set_on_close() block form methods do not work on Mac. (The webpage is never ready when Ruby executes the block.)

driven

Improved my 'GreaseKit' UserScript using a file from Dan (cheers)
with safari Css style on input borders

// ==UserScript==
// @name           inputPatch
// @author	       driven
// @namespace      http://drivenupthewall
// @description    fixes webkit input boxes in SketchUp.
// @include        *
// ==/UserScript==

(function() {	
	function main() {
// ====== DO NOT TOUCH ANYTHING ABOVE =====
// ===== PLACE CUSTOM CODE UNDER THIS LINE =====
		function patchStyle(css) { 
   var head, style; 
   head = document.getElementsByTagName('head')[0]; 
   if (!head) { return; } 
   style = document.createElement('style'); 
   style.type = 'text/css'; 
   style.innerHTML = css; 
   head.appendChild(style); 

} 

patchStyle('input;not([type=button]);not([type=file]);not([type=checkbox]);not([type=image]);not([type=radio]);not([type=reset]);not([type=search]);not([type=submit]){background-color; rgb(255, 255, 255); border-bottom-color; rgb(238, 238, 238); border-bottom-style; inset; border-bottom-width; 1px; border-collapse; separate; border-left-color; rgb(238, 238, 238); border-left-style; inset; border-left-width; 1px; border-right-color; rgb(238, 238, 238); border-right-style; inset; border-right-width; 1px; border-top-color; rgb(238, 238, 238); border-top-style; inset; border-top-width; 1px;}');
// ===== KEEP CUSTOM CODE ABOVE THIS LINE =====		
// ===== DO NOT TOUCH ANYTHING BELOW =====
	};
	var script = document.createElement('script');
	script.appendChild(document.createTextNode('('+ main + ') ();'));
	(document.body || document.head || document.documentElement).appendChild(script);
})();

john

driven

working with Safari 6[dev] as main browser
I added GreaseKit to SketchUp Menu and wrote a userScript to intercept ALL WebDialogs to add the patch.

It only affects input[type=text] even when it's not been declared. (e.g. at 3D Warehouse site)

Here are some instructions and my userScript if anyones interested.

You need to instal SIMBL and GreaseKit in these locations
/Library/Application\ Support/SIMBL
%(#4040BF)[/Library/Application\ Support/SIMBL/Plugins/GreaseKit.bundle
~/Library/Application Support/GreaseKit/apps.plist
~/Library/Application Support/GreaseKit/config.xml]
~/Library/Application\ Support/GreaseKit/inputPatch.user.js

Edit:SCRIPT REMOVED -better version below

add SketchUp to greaseKit from Safari Menu, then in SU Reload All UserScripts and Select 'InputPatch'
that's it
john

Aerilius

You could carefully "alias" the WebDialog class in SketchUp, that means you add a script that is loaded first where you redefine the UI::WebDialog.open method so that it injects your JavaScript after the WebDialog has opened. Maybe like (untested):

class UI;;WebDialog

  # this gives a new name to the method "show"
  alias ;show_orig ;show
  private ;show_orig

  def show(&proc)
    # this calls the original "show" method
    show_orig(&proc){ proc.call }
    execute_script(" /*your script*/ ")
  end

end

However re-defining base classes is something that no published plugin should do.

troyhome

@petersaal said:

I found the same problem last week. The source of the problem is the Safari update that Apple pushed out last week (5.1.7). Here's a work-around for dynamic components: make the following edits in red to the manager.css and configurator.css files. (located in Library/Application Support/Google SketchUp 8/SketchUp/Tools/Dynamic Components/css)

In Manager.css:

.edit-field {
position: absolute;
border-top: 2px solid #3063C9;
border-left: 2px solid #3063C9;
border-right: 2px solid #A5C4FF;
border-bottom: 2px solid #A5C4FF;
background-color: white; font-size: 12px;
font-family: sans-serif;
width: 100%;
height: 100%;
overflow: hidden;
padding-left: 2px;
padding-top: 1px;
border: 1px solid #000000;
}

In Configurator.css:

.config-field {
width: 100%;
font-size: 11px;
font-family: tahoma, sans-serif;
height: 22px;
background-color: white;
border: 1px solid #000000;
}

.config-field-readonly {
width: 100%;
font-size: 11px;
font-family: tahoma, sans-serif;
height: 22px;
background-color: threedface;
border: 1px solid #000000;
}

Thank you!! this worked for me!!!