internet explorer (I know I know..)

[Dimosthenis]

Thanks for this layout plugin!
Tried to run it on ie10 and edge and failed on both. Haven't checked details yet but on ie10 the console throws an error and on edge it just won't work. Will come back with more info if possible.

Any ideas?

[shchukin]

Same here.

Can be considered to use fallback like so:

.bricklayer > * { width: 150px; }

This selector will work only when plugin is disabled or not working.

[shchukin]

Maybe should update a readme?
This plugin works seamlessly with, Safari, Firefox, Chrome and all other modern browsers.

[f]

We didn't tested on IE but I did not expect that it would fail on IE Edge.

We'll test it first and we'll update it I think.

_Fatih Kadir Akın_http://github.com/f
http://twitter.com/fkadev

[Loac-fr]

If it can help, both IE10 and 11 throw an error here (line 132) :

Container.prototype.getElementsInOrder = function () {
    return this.element.querySelectorAll(":scope > *:not(." + this.options.columnClassName + "):not(." + this.options.rulerClassName + ")");
};

It's probably the same for Edge.

Error is :
SCRIPT5022 syntax error - line 132 character 13

They are supposed to handle querySelectorAll so it might be the expression ?

Edit
More specifically the issue seems to be linked with the use of :scope
See https://developer.mozilla.org/en-US/docs/Web/CSS/:scope#Browser_compatibility

[evoactivity]

Is :scope even needed there?

[f]

@evoactivity Unfortunately, yes. We need it. You have to use :scope to use > selector.

If you find a better way to make it let's do it.

[Dimosthenis]

Needs a lot of testing but managed to make it work on ie 10.

In bricklayer.js line 14
" if (window["CustomEvent"]) {.. "
changed it to
" if (window["CustomEvent"] instanceof Event) { "

and on line 123 and line 132
changed " :scope " to " .bricklayer "

Its not the best solution if someone wants to change classes but its something

[RobSchoenaker]

This appears to work for all browsers:

            Container.prototype.getColumns = function () {
                return this.element.querySelectorAll("." + this.options.columnClassName);

            };

            Container.prototype.getElementsInOrder = function () {
                return $(this.element).children().select("*:not(." + this.options.columnClassName + "):not(." + this.options.rulerClassName + ")");
            };

Keep in mind that the :scope selector is a candidate or deletion. It's an experimental selector.

[f]

May you open a PR?

8 Haz 2016 Çar 14:47 tarihinde RobSchoenaker notifications@github.com
şunu yazdı:

This appears to work for all browsers:

        Container.prototype.getColumns = function () {
            return this.element.querySelectorAll("." + this.options.columnClassName);

        };

        Container.prototype.getElementsInOrder = function () {
            return $(this.element).children().select("*:not(." + this.options.columnClassName + "):not(." + this.options.rulerClassName + ")");
        };

—
You are receiving this because you commented.

Reply to this email directly, view it on GitHub
#13 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAL_fbdnIs11V2hZBWCOpjr4_pwBOZvfks5qJqvqgaJpZM4Ibb9v
.

_Fatih Kadir Akın_http://github.com/f
http://twitter.com/fkadev

[RobSchoenaker]

I am not so good with GitHub, but I tried to create a PR. Did not work for now.. Also Typescript is not my language, but I suppose your fix is like below. Feel free to take credit.

in bircklayer.ts

private getColumns() {
  return this.element.querySelectorAll(`.${this.options.columnClassName}`)
}

private getElementsInOrder() {
  return $(this.element).children().select(`*:not(.${this.options.columnClassName}):not(.${this.options.rulerClassName})`)
}

[evoactivity]

I can confirm that the fix @RobSchoenaker suggested doesn't actually work.

I'm also having issues with bricklayer and any version of IE, not just old versions, it's one things to say "other modern browsers" to suggest older IE's wont work, but literally no version of IE works with bricklayer. That really needs to be made explicitly clear in the readme. I'm going to see what I can do as it seems related to the :scope selector but I don't understand why bricklayer needs this at all.

[evoactivity]

Ok, so using this https://github.com/lazd/scopedQuerySelectorShim and always using the custom event fallback works in ie10+

[RobSchoenaker]

@evoactivity The fix I suggested works in my (maybe specific) case. Can you PR your update to @f so we can all benefit from this?

[f]

As I understand using the shim externally will solve the problem. Means no need to change codebase. Right @evoactivity?

[evoactivity]

@RobSchoenaker When I applied your fix it caused issues fetching the correct blocks and broke the layout, will have another look at that when I have some time in the next few days.

@f pretty much yeah, the shim solves the problem for the most part, but the custom event fallback wasn't working in IE, I simply commented out the check to look for the existence of CustomEvent and always use the fallback. I'm not sure why the check didn't work, I was on a tight deadline so didn't have the time to properly investigate, again will look into it when I have time.