docker icon indicating copy to clipboard operation
docker copied to clipboard

themes not working

Open blackstormlab opened this issue 2 years ago • 18 comments

i followed the info on https://docs.phpmyadmin.net/en/latest/setup.html#:~:text=Running%20with%20additional%20themes%3A

yet when i login the theme does not show up

this is my bind

image

im using the default phpmyadmin image

image

blackstormlab avatar Mar 03 '23 01:03 blackstormlab

Which theme(s) are you trying to add? I wonder if they are compatible with your phpMyAdmin version.

ibennetch avatar Mar 03 '23 01:03 ibennetch

Blueberry theme from the official repo version of php is 5.2.1 and theme is for 5.2

blackstormlab avatar Mar 03 '23 03:03 blackstormlab

Screenshot_20230303_040914_Chrome

blackstormlab avatar Mar 03 '23 03:03 blackstormlab

Screenshot_20230303_041444_Chrome

blackstormlab avatar Mar 03 '23 03:03 blackstormlab

would there be any news on this yet?

blackstormlab avatar Mar 05 '23 02:03 blackstormlab

would there be any news on this yet?

Yes, I found what's wrong re-reading your post. The bind folder is wrong, see here the root: https://github.com/phpmyadmin/docker/blob/88f6d787d06c81de6a7cecdd59c1f788dd7c1217/apache/Dockerfile#L106

So it should be mounted to /var/www/html/themes/mytheme

williamdes avatar Mar 05 '23 02:03 williamdes

would this not mean the documentation is wrong?

blackstormlab avatar Mar 05 '23 02:03 blackstormlab

i can confirm the updated info does work

blackstormlab avatar Mar 05 '23 03:03 blackstormlab

would this not mean the documentation is wrong?

What documentation? I did not find this on the README, I will add how to mount themes

williamdes avatar Mar 05 '23 10:03 williamdes

@williamdes We have some documentation on mounting themes at

  • https://docs.phpmyadmin.net/en/latest/setup.html#docker-volumes
  • https://docs.phpmyadmin.net/en/latest/setup.html#docker-examples and
  • https://docs.phpmyadmin.net/en/latest/setup.html#customizing-configuration-file-using-docker-compose

ibennetch avatar Mar 05 '23 14:03 ibennetch

Thank you, that's a very strange folder for themes. Did it work someday? I need to research that

williamdes avatar Mar 05 '23 16:03 williamdes

@williamdes We have some documentation on mounting themes at

  • https://docs.phpmyadmin.net/en/latest/setup.html#docker-volumes
  • https://docs.phpmyadmin.net/en/latest/setup.html#docker-examples and
  • https://docs.phpmyadmin.net/en/latest/setup.html#customizing-configuration-file-using-docker-compose

Those are indeed the docs I followed.

blackstormlab avatar Mar 05 '23 16:03 blackstormlab

Seems to have been since https://github.com/phpmyadmin/docker/issues/203#issue-392540166 and here is another post where someone also used this mount path: https://github.com/phpmyadmin/docker/issues/333#issuecomment-863560724

So I found out it's since https://github.com/phpmyadmin/docker/commit/662b82d27e87722826ce4c40d2db83553e79c898 (#198)

The best is that we now update our documentation

williamdes avatar Mar 09 '23 21:03 williamdes

@tianon on the same subject as this issue, with #198 & #274 we no longer can mount sessions to /sessions I found a message of yours about that in: https://github.com/docker-library/php/issues/201#issuecomment-243233534

Can we create a new folder /sessions and change the php ini config to this folder or is it not a good change for an official image ? /cc @J0WI Currently session.save_path => no value, would be changed to session.save_path => /sessions/ What would be the best permissions for such a folder ?, for rootless compatibility ?

williamdes avatar Mar 09 '23 21:03 williamdes

If this makes sense for how you want phpMyAdmin users to consume the image, it's totally fine (we just don't feel comfortable make such a general sweeping change in the php images directly).

If you want rootless compatibility, I'd suggest 1777 (matching typical /tmp permissions).

tianon avatar Mar 10 '23 23:03 tianon

Alternatively, you could configure an external Redis or Memcached container as PHP session handler.

J0WI avatar Mar 21 '23 20:03 J0WI