Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Encrypted Field's Label settings don't change #742

Open
l-mittscherlich opened this issue Oct 9, 2024 · 4 comments · Fixed by #735
Open

[Bug]: Encrypted Field's Label settings don't change #742

l-mittscherlich opened this issue Oct 9, 2024 · 4 comments · Fixed by #735
Assignees
Milestone

Comments

@l-mittscherlich
Copy link

Pimcore version

v11.3.2.1

Steps to reproduce

Create a Panel
Create an Encrypted Field inside of the Panel
Change Panels Label Width/Align settings

Actual Behavior

Label Width doesn't change
Label Align on value 'top' is still left from input field

Expected Behavior

Label Width should change dependent on value in Label Width
Label Align should change the Labels position

Copy link

Thanks a lot for reporting the issue. We did not consider the issue as "Pimcore:Priority", "Pimcore:ToDo" or "Pimcore:Backlog", so we're not going to work on that anytime soon. Please create a pull request to fix the issue if this is a bug report. We'll then review it as quickly as possible. If you're interested in contributing a feature, please contact us first here before creating a pull request. We'll then decide whether we'd accept it or not. Thanks for your understanding.

@blankse
Copy link
Contributor

blankse commented Oct 29, 2024

@l-mittscherlich I created a PR: #735

@mcop1 mcop1 transferred this issue from pimcore/pimcore Nov 8, 2024
@mcop1 mcop1 added this to the 1.6.2 milestone Nov 8, 2024
@mcop1 mcop1 linked a pull request Nov 8, 2024 that will close this issue
@mcop1 mcop1 self-assigned this Nov 8, 2024
@mcop1
Copy link
Contributor

mcop1 commented Nov 8, 2024

I moved the ticket from pimcore to admin-ui repository.
Thanks for your efforts guys, we appreciate it.

@blankse
Copy link
Contributor

blankse commented Nov 8, 2024

Fixed by #735

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

Successfully merging a pull request may close this issue.

3 participants