MODX :: Anyone successfully using QuickEdit and IE7?

788

Jesse R.
Consider trying something new and extraordinary.
Illinois Wine

Have you considered donating to MODx lately?
Donate now. Every contribution helps.

jesster444 Reply #1, 4 years, 11 months ago

Link to this post#1

I am going to try and word my question a little different. Is anyone able to get QuickEdit and IE7 working? If so, what version are you using? If you are using 0.9.6, what revision?

14,561

How MODx Evo Works Log in to an Evo Manager username guest, password guestuser.

sottwell Reply #2, 4 years, 11 months ago

Link to this post#2

This has been brought up a few times already; it seems to have someting to do with some code for handling transparencies somewhere. I'm not sure; try a search to get the other posts on the subject.

788

Jesse R.
Consider trying something new and extraordinary.
Illinois Wine

Have you considered donating to MODx lately?
Donate now. Every contribution helps.

jesster444 Reply #3, 4 years, 11 months ago

Link to this post#3

Actually, I am getting javascript errors, so I don't think it relates to the CSS problems mentioned before.

Particularly 'qe' is undefined when clicking the "edit xxxxx" buttons, and 'toolbar' undefined when the page loads.

30

StainlessSteelRat Reply #4, 4 years, 10 months ago

Link to this post#4

I'm having the same issue. Any ideas what's causing it? (I get the same thing in IE7 and IE6.)
I'm currently at 0.9.6-RC2 (latest SVN build to see if that would fix it), though the issue has been there since before RC1.

788

Jesse R.
Consider trying something new and extraordinary.
Illinois Wine

Have you considered donating to MODx lately?
Donate now. Every contribution helps.

jesster444 Reply #5, 4 years, 10 months ago

Link to this post#5

I am sorry to bump this as it is really annoying some of my users that use IE7 only at work. And now someone else is reporting the same issue. I would have to imagine that more than two people have come across this. Is this a bug, or some configuration problem? ::Looks in Ryan's and Jeff's direction::

30

StainlessSteelRat Reply #6, 4 years, 10 months ago

Link to this post#6

Yes... my users aren't too happy about it either. BTW... I searched & tried the "fix" found in other postings about removing the alpha filter references from toolbar.css, but I couldn't find any.

788

Jesse R.
Consider trying something new and extraordinary.
Illinois Wine

Have you considered donating to MODx lately?
Donate now. Every contribution helps.

jesster444 Reply #7, 4 years, 10 months ago

Link to this post#7

Plus this appears to be a javascript problem and not a CSS problem.

23

Seraji Reply #8, 4 years, 10 months ago

Link to this post#8

Having the exact same problem here in both IE6 and 7. This problem is isolated to the new mootools driven one and only ever applies to IE, Firebug cannot detect the error.

What i get is that the toolbar appears (added to page by php/js on load?) but then it is impossible to use it at all, the error recieved on the toolbar is "toolbar is undefined, line 76 char 3" this code is

74.// Draggable toolbar title
75.  this.toolbar.makeDraggable({
76.   handle: $E('h1',toolbar),
77.  onStart:function() { this.toolbar.setStyle('opacity',0.5); }.bind(this),
78.   onComplete: function(x,y) { this.toolbar.setStyle('opacity',1);Cookie.set('QE_position', this.toolbar.getLeft()+'/'+this.toolbar.getTop(), {duration:this.cookieDuration}); }.bind(this)
  });

The first reference to 'toolbar' is several lines up and is

this.toolbar = toolbar;
  this.menu = $E('ul',toolbar);
  this.title = $E('h1',toolbar);
  this.buttons = $ES('li','QE_Toolbar');

which is a declaration of what toolbar is, a definition if you like, which is why FF Safari (webkit) both recognize it.
But that doesnt make sense as being the problem and here is why, from the IE developer toolbar (aka. the 'srsly why doesnt this work in IE toolbar')

/* Inline style */ {	DISPLAY: block}

That inline styling is applied by the javascript here

 this.toolbar.setStyle('display','block');

so that earlier declaration works here as the js is happily displaying the toolbar for us to be tricked into thinking it works.

So, lets play spot the difference, every other call to 'toolbar' is done by 'this.toolbar' only the broken one uses toolbar by itself, this is cause this.toolbar=toolbar defines this.toolbar as something, not just toolbar by itself. No idea why FF could sort that out itself but at least now it functions in IE7 (ill move to IE 6 later).

The dropdowns worked fine after that change and so did dragging, however once you dragged the bar the dropdowns ceased to display (note: the height of the menu's still changed, JS still working fine) something about how the opacity was throwing off IE7, so anyway removed the two lines about opacity, now the bar works all the time in IE7, FF and Sarafi.

Also that seems to have fixed the little edit content error (qe is not blah blah) that i also was getting.

IE 6 the bar moves but none of the dropdowns are visible and you cant click them, probably to do with css z-index as IE has issues dealing with that. They have displayed in that way since i upgraded, ill look into it when i get some more time, right now im just pleased that IE 7 can use the bar again.

Anyhow I've attached the new js to this post and its the only file i edited and tested it on my clean install of 0.9.6 SVN so i'd say its fine.

Edit: Fix for IE6 is to edit toolbar.css and cut the width of h1 down to 110px from 215px, in IE6 the faulty z-index rendering meant that h1 simply refused to let the menu show through at all.

443

g5604 Reply #9, 4 years, 10 months ago

Link to this post#9

check this thread for the fix:

http://modxcms.com/forums/index.php?PHPSESSID=37c8e9064cf99697ecfc4cd9e393c514&/topic,10200.msg70442.html

788

Jesse R.
Consider trying something new and extraordinary.
Illinois Wine

Have you considered donating to MODx lately?
Donate now. Every contribution helps.