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

Custom themes not working in 5.2 #379

Closed
ToshY opened this issue Aug 1, 2022 · 5 comments
Closed

Custom themes not working in 5.2 #379

ToshY opened this issue Aug 1, 2022 · 5 comments
Assignees
Labels

Comments

@ToshY
Copy link

ToshY commented Aug 1, 2022

Problem

Since phpMyAdmin 5.2 it doesn't seem possible to mount volume for a custom themes anymore.

The documentation for docker examples in the main repo makes this even more confusing as mounting to /www/themes doesn't seem to have any effect, as where mounting to /var/www/html/themes worked for 5.1.

Reproduction

  1. Download a theme from themes repo, e.g. darkwolf.
  2. Mount theme directory to /www/themes
  pma:
    image: phpmyadmin:5.2
    security_opt:
      - no-new-privileges:true
    restart: always
    environment:
      DATABASE_ALLOW_NO_PASSWORD: 0
      PMA_HOST: ${DATABASE_HOST}
      PMA_PORT: ${DATABASE_PORT}
      HIDE_PHP_VERSION: 1
      PMA_ABSOLUTE_URI: "https://pma.example.local"
    networks:
      - proxy
    volumes:
      - pma_data:/sessions
      - ./docker/pma/themes/darkwolf:/www/themes/darkwolf
  1. Take a look at the homepage

image

Mounting to /var/www/html/themes doesn't show the new theme in 5.2 either.

@williamdes
Copy link
Member

Hi,
Are you sure to have a 5.2 compatible theme?
Can you check in docker/pma/themes/darkwolf/theme.json?

@ToshY
Copy link
Author

ToshY commented Aug 1, 2022

Hey @williamdes 👋

Thanks for the quick response. Just noticed that the theme.json says versions 5.0 and 5.1. After manually adding 5.2 to it, I was able to use it in the dashboard, although it was indeed not quite compatible yet 🤦‍♂️

With that mystery solved, maybe it's still a good idea to update the docker documentation part for themes to change the mounting from /www/themes to /var/www/html/themes then?

Also, what is the actual difference between phpmyadmin/phpmyadmin and phpmyadmin images, as both are used seemingly interchangeable in the docker examples documentation here and here?

@williamdes
Copy link
Member

williamdes commented Aug 1, 2022

Hey @williamdes 👋

Hey 👋🏻

Thanks for the quick response. Just noticed that the theme.json says versions 5.0 and 5.1. After manually adding 5.2 to it, I was able to use it in the dashboard, although it was indeed not quite compatible yet 🤦‍♂️

You can try the zip phpmyadmin/themes#78 and provide feedback to the author :)

With that mystery solved, maybe it's still a good idea to update the docker documentation part for themes to change the mounting from /www/themes to /var/www/html/themes then?

Why, the documentation is wrong?

Also, what is the actual difference between phpmyadmin/phpmyadmin and phpmyadmin images, as both are used seemingly interchangeable in the docker examples documentation here and here?

Maybe you are talking about what I fixed yesterday: phpmyadmin/phpmyadmin@8ef83c0

Edit: I need to fix this command line

Also I think you may want to read #363

@williamdes williamdes self-assigned this Aug 1, 2022
@ToshY
Copy link
Author

ToshY commented Aug 1, 2022

You can try the zip phpmyadmin/themes#78 and provide feedback to the author :)

Good idea 👍

Why, the documentation is wrong?

In the Docker Examples section, the last example for adding themes shows the following

docker run --name phpmyadmin -d --link mysql_db_server:db -p 8080:80 -v /custom/phpmyadmin/theme/:/www/themes/theme/ phpmyadmin/phpmyadmin

which mounts to /www/themes/theme. So change it to /var/www/html/themes/theme then?

Also I think you may want to read #363

Yes, that clears things up. Thanks.

williamdes added a commit to phpmyadmin/phpmyadmin that referenced this issue Aug 9, 2022
And use the same container name everywhere

Seems like I missed some entries

Ref: 249d02f
Ref: phpmyadmin/docker#56
Ref: phpmyadmin/docker#379

Signed-off-by: William Desportes <williamdes@wdes.fr>
williamdes added a commit to phpmyadmin/phpmyadmin that referenced this issue Aug 9, 2022
Closes: phpmyadmin/docker#379

Signed-off-by: William Desportes <williamdes@wdes.fr>
@williamdes
Copy link
Member

which mounts to /www/themes/theme. So change it to /var/www/html/themes/theme then?

Indeed, it seems like other users figured it out too (#367 (comment))

I pushed phpmyadmin/phpmyadmin@7faf6a5 to fix the documentation

ankush-0x90 pushed a commit to ankush-0x90/phpmyadmin that referenced this issue Sep 6, 2022
And use the same container name everywhere

Seems like I missed some entries

Ref: 249d02f
Ref: phpmyadmin/docker#56
Ref: phpmyadmin/docker#379

Signed-off-by: William Desportes <williamdes@wdes.fr>
ankush-0x90 pushed a commit to ankush-0x90/phpmyadmin that referenced this issue Sep 6, 2022
Closes: phpmyadmin/docker#379

Signed-off-by: William Desportes <williamdes@wdes.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants