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

Template shortcode field size, name attribute #145

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Template shortcode field size, name attribute #145

wants to merge 4 commits into from

Conversation

sarojlakra
Copy link
Contributor

Template shortcode input field in the builder area is too small and shortcode is fully not visible. making width:auto would be great

template_shortcode_field

template_shortcode_field_width_auto

Template shortcode input field in the builder area is too small and shortcode is fully not visible. making width:auto would be great
@sarojlakra sarojlakra changed the title Template shortcode field size Template shortcode field size, name attribute Jul 20, 2014
@syamilmj
Copy link
Owner

Hey good call on the template name in the shortcode. I think to make it consistent, we should also include the template name inside the shortcode field provided on the panel.

This also means that setting width: auto, would potentially break things when the template names are longer.. In which case, I think setting a max-width value as well should be good.

Anyway, I just pushed a bunch of commits that conflict with this, so would you mind fetch & rebase on your machine, fix any conflicts and send another PR?

Thanks man, appreciate the contributions! :)

@sarojlakra
Copy link
Contributor Author

Updated: #149

Thanks

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

Successfully merging this pull request may close these issues.

2 participants