Skip to content
This repository has been archived by the owner on Mar 8, 2021. It is now read-only.

Some minor issues with 1.2-RC1 #888

Closed
fourroses666 opened this issue Oct 31, 2016 · 26 comments
Closed

Some minor issues with 1.2-RC1 #888

fourroses666 opened this issue Oct 31, 2016 · 26 comments

Comments

@fourroses666
Copy link
Contributor

1.2-RC1

just updated a site and missing last 2 tabs

missing-tab

@fourroses666 fourroses666 changed the title missing tab Missing two last Config tabs Oct 31, 2016
@fourroses666 fourroses666 changed the title Missing two last Config tabs Missing the two last Config tabs Oct 31, 2016
@fourroses666 fourroses666 reopened this Oct 31, 2016
@Dmi3yy
Copy link
Collaborator

Dmi3yy commented Oct 31, 2016

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 (#888), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAo3M0CjhVSmQmhnhOcAZCgm8d2_W5GUks5q5lkNgaJpZM4KlgiE).

@fourroses666
Copy link
Contributor Author

Yea correct.

@fourroses666
Copy link
Contributor Author

fourroses666 commented Oct 31, 2016

  • 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?

@Dmi3yy
Copy link
Collaborator

Dmi3yy commented Oct 31, 2016

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 (#888 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAo3M-DgI5rwzkrebwDXpj6ViuBTIvVuks5q5l9JgaJpZM4KlgiE).

@fourroses666
Copy link
Contributor Author

correct!

and file manager location not correct its saying

@fourroses666
Copy link
Contributor Author

Ah, had to re refresh manager path. My bad

@fourroses666
Copy link
Contributor Author

fourroses666 commented Oct 31, 2016

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
Copy link
Contributor

Deesen commented Nov 1, 2016

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
Copy link
Contributor

Deesen commented Nov 1, 2016

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
Copy link
Contributor

Deesen commented Nov 1, 2016

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
Copy link
Contributor Author

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
Copy link
Contributor Author

fourroses666 commented Nov 1, 2016

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
Copy link
Contributor

Deesen commented Nov 1, 2016

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

@Deesen
Copy link
Contributor

Deesen commented Nov 1, 2016

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

@fourroses666 fourroses666 changed the title Missing the two last Config tabs Some minor issues with 1.2-RC1 Nov 1, 2016
@fourroses666
Copy link
Contributor Author

fourroses666 commented Nov 1, 2016

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 fourroses666 reopened this Nov 1, 2016
@fourroses666
Copy link
Contributor Author

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
Copy link
Contributor Author

fourroses666 commented Nov 1, 2016

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
Copy link
Contributor

pmfx commented Nov 1, 2016

@fourroses666 #895

@fourroses666
Copy link
Contributor Author

fourroses666 commented Nov 1, 2016

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
Copy link
Contributor

Deesen commented Nov 3, 2016

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

@fourroses666
Copy link
Contributor Author

fourroses666 commented Nov 4, 2016

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
Copy link
Contributor

Deesen commented Nov 4, 2016

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

@fourroses666
Copy link
Contributor Author

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
Copy link
Contributor Author

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
Copy link
Contributor Author

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
Copy link
Contributor

Deesen commented Nov 9, 2016

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!

Dmi3yy pushed a commit that referenced this issue Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants