modxcms / evolution

This repository was Frozen. Welcome to the new evolution of MODX Evolution!
https://github.com/evolution-cms/
178 stars 207 forks source link

Some minor issues with 1.2-RC1 #888

Closed fourroses666 closed 7 years ago

fourroses666 commented 7 years ago

1.2-RC1

just updated a site and missing last 2 tabs

missing-tab

Dmi3yy commented 7 years ago

turn off tinyMCE plugin

Lukianenko Dmytro Web developer
Site: http://dmi3yy.com Skype: dmi3yy Phone: +38 (091) 357-10-80

понедельник, 31 октября 2016 г. в 23:09, Marc написал:

1.2-RC1 just updated a site and missing last 2 tabs

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub (https://github.com/modxcms/evolution/issues/888), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAo3M0CjhVSmQmhnhOcAZCgm8d2_W5GUks5q5lkNgaJpZM4KlgiE).

fourroses666 commented 7 years ago

Yea correct.

fourroses666 commented 7 years ago

Images are there, but not visible, or is it some server issue?

Dmi3yy commented 7 years ago

check assets/cache/images/.htaccess

Lukianenko Dmytro Web developer
Site: http://dmi3yy.com Skype: dmi3yy Phone: +38 (091) 357-10-80

понедельник, 31 октября 2016 г. в 23:36, Marc написал:

Can't reach File Manager. Images not visible, like; /assets/cache/images/banner/640x-banner2.42d.png

Images are there but not visible, or is it some server issue?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub (https://github.com/modxcms/evolution/issues/888#issuecomment-257427653), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAo3M-DgI5rwzkrebwDXpj6ViuBTIvVuks5q5l9JgaJpZM4KlgiE).

fourroses666 commented 7 years ago

correct!

and file manager location not correct its saying

fourroses666 commented 7 years ago

Ah, had to re refresh manager path. My bad

fourroses666 commented 7 years ago

Don't know if this is a bug:

No output:

[*id:is=`5`:then=`[[Ditto? 
&parents=`5`
&total=`all` 
&showPublishedOnly=`1`
&showInMenuOnly=`0`
&orderBy=`menuindex ASC` 
&noResults=` ` 
&tpl=`tpl-blog-overzicht`
]] `:else=``*]

After a space before [[Ditto.. it does work: [*id:is=5:then= [[Ditto?`


Very small detail; File Manager folders has a larger line-height?: Don't know why.

detail

Deesen commented 7 years ago

Reg file-manager its strange indeed. I also could not find out where line-height comes from.

Reg. [*id...*] I think its a bug. I added another [*id:is*] with only "yes" or "no" as result but no output. There is only output if I remove the [*id:then='[[Ditto...

My testcase using demo-site is:

[*id*]<br/>
[*id:is=`49`:then=`yes`:else=`no`*] <br/>
[*id:is=`49`:then=`[[Ditto?
&parents=`2`
&total=`all` 
&showPublishedOnly=`1`
&showInMenuOnly=`0`
&orderBy=`menuindex ASC` 
&noResults=`no results` 
&tpl=`ditto_blog`
]]`:else=`else`*]
Deesen commented 7 years ago

If I add table tr td { height:21px; } to file-manager all row have height 27px. Setting it below 21px there is always some difference like .5px.

Deesen commented 7 years ago

I found the reason. Using a snippet call is ok as the snippet-result / return will be rendered first. Means

[*id:is=`49`:then=`[[Ditto?&tpl='ditto_tpl']]`:else=``*]

will be send to the "is/then/else"-mechanismn like

[*id:is=`49`:then=`<div class="blogentry">... [!jot?&param=`xy`!] ...</div>`:else=``*]

The first sign after then= is the delimiter like " or ' or ´. In this case it is ´. If {{ditto_tpl}} now outputs this delimiter-sign like within above snippet-call [[jot]], the filter/modifier-mechanismn gets broken as it will be handled as

then=`<div class="blogentry">... [!jot?&param=`

It will search for the next : to determine the else and so on...

fourroses666 commented 7 years ago

Probably it works when using a chunk instead of the direct snippet call and maybe if? works too. But I don't understand how can it be possible when I add a simple blank space it does work.

fourroses666 commented 7 years ago

Reg. the space, I think the image isn't a block/inline-block and adds a 5px at the bottom. Can't look into it right now.

Deesen commented 7 years ago

You are right, for me even the space(s) did not work. I have no clue..

Deesen commented 7 years ago

Maybe you can post what the Ditto-snippet itself returns so I can see its exact output.

fourroses666 commented 7 years ago

Oke, will do. Can't at the moment.

Have another small "issue" I just noticed. eForm is outputting inputfields before checkboxes/radiobuttons errors . Maybe this sortorder can be fixed?

order-eform

fourroses666 commented 7 years ago

Need the space to show the ditto.

            [*id:is=`5`:then=` [[Ditto? 
                &parents=`5`
                &total=`all` 
                &showPublishedOnly=`1`
                &showInMenuOnly=`0`
                &orderBy=`menuindex ASC` 
                &noResults=` ` 
                &tpl=`tpl-blog-overzicht`
            ]]`:else=``*]

Chunk:

<div class="item" style="float:left; width:20%; padding:15px;">
    <a href="[~[+id+]~]">
        <img src="[[phpthumb? &input=`[+blog-afb1+]` &options=`w=300,h=380,zc=1`]]" />
        <div><h3>[+description+]</h3></div>
    </a>
</div>
fourroses666 commented 7 years ago

Can css be changed please? For manager theme RE2

For MAC and firefox we need this style:

*{
-moz-osx-font-smoothing: grayscale;
}

Change password and logout top right are too "bright".

#supplementalNav a {
    color: rgba(255, 255, 255, 0.75) !important;
}

Codemirror font-size is a bit big/uge. /plugins/codemirror/cm/lib/codemirror.css

.CodeMirror {
    font-size: 1.1em;
much nicer ---> font-size: 1em;
}
pmfx commented 7 years ago

@fourroses666 #895

fourroses666 commented 7 years ago

Thanksss,

See some small issue with dutch lang but don't think there is an easy solution. Probably need to translate that button. Or make it an inline-block but no clear:both after the refreshbutton so still looks weird.

dutch-lang

"Nieuwe" can be removed.

But thats just some detail. Not really important..

Maybe remove the left padding on these buttons, could be done with CSS but not sure if it will give issues with other buttons. #treePane .tab-page ul li:first-child padding-left:0;}

Deesen commented 7 years ago

Reg. "space to show ditto" I used your chunk but no change. Even with space I have no output.

fourroses666 commented 7 years ago

Hmm, thats weird. It really does work with space (or character) with me. Maybe its not meant to use snippets into the "parameters?". (Or my PHx plugin is different?) Probably will work when using a chunk for the Ditto call. Haven't tried changing it to DocLister actually. Maybe it does with that.

-edit- Just tested DocList but that didn't work too. With a space it did. Will keep it this way with a space but if you would like to have a look its possible to give access @Deesen (its on a updated site not fresh install)

Deesen commented 7 years ago

Your PHx-plugin? I tried it using the new built-in filters, not PHx.

fourroses666 commented 7 years ago

Ah yea, thought [*id:is=``... uses PHx. Not sure it did overwrite my PHx settings plugin on Evo update which could be different on my basic installs. Anyway just trying to find the differences why it does work on my install and yours not, its weird.

Hope to do some more testing this weekend on the latest build to see if there are any issues/bugs.

fourroses666 commented 7 years ago

Don't know if its already fixed but with the "elements tree" my TV's and (if I'm correct) Plugins are not visible. You can also see it here on this screendump:

dutch-lang

There are no TV's :o

fourroses666 commented 7 years ago

I really like the new RC2 theme, no doubt. I do see there can be done many small design improvements.

The dashboard uses font-size:13px; The manager resources uses font-size:75%; TinyMCE (4) uses font-size:15px;

I have the best readability when using 10pt. All these should be font-size:13px or 10pt.

I would have added some pull requests if I had some time to do it.

When I have some spare time ( and internet at home, broken router 👎 ) I would like to make a new style for TinyMCE4 and also the KCFinder to look like it belongs to the manager theme.

Deesen commented 7 years ago

Are your TVs back like mentioned in https://github.com/pmfx/ElementsInTree/issues/19?

The theme is indeed great :-). Reg. font-sizes, I leave that up to you guys. But an exclusive Tiny4-theme / KCFinder-theme sounds nice!