Skip to content
This repository has been archived by the owner on Oct 6, 2019. It is now read-only.

Show Content Pane Labels #1

Closed
ohine opened this issue Aug 12, 2018 · 3 comments · Fixed by #19
Closed

Show Content Pane Labels #1

ohine opened this issue Aug 12, 2018 · 3 comments · Fixed by #19
Labels
Milestone

Comments

@ohine
Copy link
Contributor

ohine commented Aug 12, 2018

Description

When in edit mode > hover over a module > there is a Move flyout menu with content panes listed to reposition the module on a page.

Admins need a way to easily identify the name of each content pane in order for this to be a usable feature.

Originally created by @tpluscode at dnnsoftware/Dnn.EditBar#25

@ohine
Copy link
Contributor Author

ohine commented Aug 12, 2018

Comment by @OllyHodgson

Recently I've worked around this by making sure all of my content panes have a data-name attribute the same as the ID, e.g. <div runat="server" id="Fullwidth_Central" data-name="Fullwidth_Central"> and used CSS to show that attribute:

.dnn-mode-edit .content-areas .dnnModuleManager::before {
    content: attr(data-name);
    display: inline-block;
    padding: 2px 3px;
    border-radius: 4px;
    position: absolute;
    z-index: 1;
    top: 12px;
    left: 12px;
    font-size: 11px;
    color: $white;
    background: rgba(0,0,0,0.3);
}

It shows up like this when editing the page (extra icons are because it's an Evoq install):

image

@ohine ohine added the esw label Aug 12, 2018
@ohine ohine added this to the 1.6.1 milestone Aug 12, 2018
@tpluscode
Copy link
Contributor

Actually, I like @OllyHodgson's code better. I would still consider hover style instead though.

@zyhfish, would you adapt your solution to use similar method to achieve this goal?

@OllyHodgson
Copy link
Contributor

OllyHodgson commented Aug 14, 2018

@tpluscode

I would still consider hover style instead though.

If DNN's editing experience is ever going to be good on touch only devices (iPad, etc) remember hover state doesn't really exist there :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants