Suggestion #27544

change divs to real extjs toolbars ;)

Added by Kay Strobach almost 2 years ago. Updated 8 months ago.

Status:Needs Feedback Start date:2011-06-19
Priority:Could have Due date:
Assignee:- % Done:

0%
Category:-
Target version:-
Votes: 0

Description

the following code should be replace with a real extjs tbar in the module menu (and perhaps in the page and file tree) to be consitent with new modules an cards ;)

    '<div id="typo3-docheader">',
    '    <div id="typo3-docheader-row1">',
    '        <div class="buttonsleft"></div>',
    '        <div class="buttonsright"></div>',
    '    </div>',
    '</div>',

Related issues

related to Usability Team - Suggestion #27432: new condensed mode (hide the left menu) Accepted 2011-06-14

History

#1
Updated by Steffen Gebert almost 2 years ago

I think that's not directly a task for the usability team - more for the core team or you ;-)

Are you fine with moving it?

#2
Updated by Kay Strobach almost 2 years ago

hi steffen, isn't the skin team better?

Anyway the config of the menu module should be extracted from the extonready section too ;)

Regards
Kay

#3
Updated by Jens Hoffmann almost 2 years ago

Hi Steffen. Move that technical thing here please to the Core/Skin Team.
I guess it's not that easy to "just" replace some parts at some places.
But that isn't a Design/Usability thing at all ... so :)

@Kay: Don't add technical Issues here, please. Ok?! :)

#4
Updated by Kay Strobach almost 2 years ago

ok, i will fix this during the rework of the module menu ...

Thanks for moving anyway (thought that it is correct here, as it is highly related).

#5
Updated by Steffen Gebert almost 2 years ago

Jens, can you move it? I always get an error message. As I'm not a team member here, maybe I'm not allowed to do so (however, then the error message is bad).

#6
Updated by Jens Hoffmann almost 2 years ago

  • Tracker changed from Bug to Suggestion
  • Project changed from Usability Team to Skin Team
  • Status changed from New to Needs Feedback
  • Priority changed from Should have to Could have

Moved form the Usability Team to the Skin Team

#7
Updated by Kay Strobach almost 2 years ago

note for me:

open file typo3/js/extjs/Viewport/viewportconfiguration.js

Search module:typo3-module-menu add property: tbar:[' ']

Then remove 

'&lt;div id="typo3-docheader"&gt;',
        '    &lt;div id="typo3-docheader-row1"&gt;',
        '        &lt;div class="buttonsleft"&gt;&lt;/div&gt;',
        '        &lt;div class="buttonsright"&gt;&lt;/div&gt;',
        '    &lt;/div&gt;',
        '&lt;/div&gt;',

from the file modulemenu.js in typo3/js ;)

Additionally tbars need to be styled properly (e.g. remove margins to bottom ...)

#8
Updated by Ernesto Baschny 10 months ago

This is probably obsolete by now. And while you are checking it, please take a look at this new suggestion #38759, which is being reviewed already. I think it looks very nice!

#9
Updated by Felix Kopp 8 months ago

Also the backend might be migrated to more jQuery-Elements therefor ExtJS toorbars might be considered contra productive.

I think this issue should be closed.

Also available in: Atom PDF