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

Background image no longer working #323

Open
mdjnelson opened this issue Feb 9, 2020 · 38 comments
Open

Background image no longer working #323

mdjnelson opened this issue Feb 9, 2020 · 38 comments

Comments

@mdjnelson
Copy link
Owner

See https://moodle.org/mod/forum/discuss.php?d=396340.

@mdjnelson
Copy link
Owner Author

@EFoster1068
Copy link

Attached is the background that I am using. I also sent a picture of my template as I was editing and you can see where it moved out of the normal position. When I start editing the position will be correct. Then when I go to view the certificate and come back to edit a couple times it will move all of the elements including the background as seen below.

Website Example 2

Website Example

Background

@emmarichardson
Copy link

Is it that the background image moves or the other elements move? My background image seems to be just fine but the other elements do not display in the position they are set to when editing. They all move slightly.

@EFoster1068
Copy link

EFoster1068 commented May 26, 2020 via email

@EFoster1068
Copy link

Any success with solving this issue?

@mdjnelson mdjnelson moved this from To do to In progress in 3.8.5, 3.7.5, 3.6.7, 3.5.9 Aug 8, 2020
@mdjnelson
Copy link
Owner Author

Looking at this now.

@mdjnelson
Copy link
Owner Author

I can not replicate this, even with the image provided. It looks and works fine for me. I think this is a theme issue. Has anyone tried changing their theme to 'Classic' or 'Boost' temporarily and seeing if that fixes the problem?

@EFoster1068
Copy link

EFoster1068 commented Sep 22, 2020

I tried it on Adaptable, Boost and Classic and had the same issue. It seems like it is not placing the background in the square that appears when you set the size. Therefore everything is shifted up about 14-20 units.
As you can see below the buttons "save and close" "save and continue" are not visible. in pic #2 Pics below are in the Classic Theme.

Certificate error 1
Certificate Error 2

@mdjnelson
Copy link
Owner Author

What width and height have you set your certificate to?

@mdjnelson
Copy link
Owner Author

II have been playing around with this for some time and can not replicate it. I added a grade and code element and moved them around and kept re-visiting the reposition page in both Boost and Classic and there were no issues.

@emmarichardson
Copy link

emmarichardson commented Oct 13, 2020 via email

@mdjnelson
Copy link
Owner Author

Can you tell me what width and height you have set for that page of the certificate?

@emmarichardson
Copy link

emmarichardson commented Oct 14, 2020 via email

@EFoster1068
Copy link

EFoster1068 commented Oct 21, 2020 via email

@EFoster1068
Copy link

EFoster1068 commented Oct 21, 2020 via email

@jardao
Copy link

jardao commented Nov 5, 2020

I have the same problem also using an image (not a background image)

@mdjnelson
Copy link
Owner Author

What browser are you all using?

@EFoster1068
Copy link

EFoster1068 commented Nov 25, 2020 via email

@mdjnelson
Copy link
Owner Author

Hm, same. I also set my width to 280 and height to 215, added a BG image, a normal image and when I press 'Save and continue' on the reposition page the images remain where they are and the buttons are not hidden by them.

@mdjnelson
Copy link
Owner Author

Testing issue 323

@mdjnelson
Copy link
Owner Author

See above. This is what I have. Are you running the latest 3.8 on github, or did you download from the plugins DB?

@EFoster1068
Copy link

EFoster1068 commented Nov 25, 2020 via email

@EFoster1068
Copy link

I have had someone work on this issue and it seems they found a solution. I would like to send the files to you so you can find what they came up with. I just need to know how to get them to you.

@mdjnelson
Copy link
Owner Author

Could they sign up to Github and put the solution in this comment section?

@EFoster1068
Copy link

@mdjnelson
Copy link
Owner Author

Thanks. It would be preferable to see just the changes that were done (in a patch) but I can do a diff and see what the differences are.

@joshwillcock
Copy link

joshwillcock commented Feb 26, 2021

Thanks. It would be preferable to see just the changes that were done (in a patch) but I can do a diff and see what the differences are.

Hey Mark.

I've not had any involvement but have been watching this issue; I tried comparing the uploaded ZIP to help out but didn't have much luck.

It appears the developer added three lines to the savepositions function in rearrange-debug.js and rearrange.js
window.scrollTo(0, 0);
document.querySelector("body").style.height = '100%';
document.querySelector("body").style.overflow = 'hidden';

Commented out the whole of rearrange-min.js and replaced it's usage by writing the JS manually in rearrange.php
This appears to be a full re-write rather than one or two small changes. I think if you can read what they wrote in rearrange.php line 135+ you might recognise something immediately different.

I was hoping originally to save you from the workload, but there is so much JS named slightly differently I didn't want to muddy the situation further.

Thanks for your hard work with this fantastic plugin!

@mdjnelson
Copy link
Owner Author

Thanks for doing research on this Josh. Much appreciated.

@mdjnelson
Copy link
Owner Author

@EFoster1068 are you able to provide a diff with the changes, or create a merge request?

@EFoster1068
Copy link

EFoster1068 commented Mar 25, 2021 via email

@estherkrisanti
Copy link

I experience the same issue. Could you please help to share what should be changed? Thanks!

@kpi6288
Copy link

kpi6288 commented Jul 12, 2022

I have the same issue using Trema design.
Worked when I changed back to the Classic design, though.

Klaus

@teruselearning
Copy link

Occurring on a Vanilla Moodle 4.0 with boost and classic themes when managing templates for me.

@shikizisykes26
Copy link

Hello, is the problem solved? what I experienced was for some certificates, the background image went up to the top to close the save and close button. I've tried changing the theme (boost and classic) but it's still the same, Please advise.
i use moodle version : 3.11
plugin version : 3.11.1 (2021051701)
theme : Edumy

@EFoster1068
Copy link

EFoster1068 commented Nov 14, 2022 via email

@EFoster1068
Copy link

EFoster1068 commented Nov 14, 2022 via email

@mdjnelson mdjnelson added this to To do in 4.1.4, 4.2.5 Mar 19, 2024
@mdjnelson mdjnelson moved this from To do to In progress in 4.1.4, 4.2.5 Mar 19, 2024
@mdjnelson
Copy link
Owner Author

Looking at this again. It's been a while, sorry.

@mdjnelson
Copy link
Owner Author

I've tried switching themes, page height and width, adding numerous items but the reposition page is fine and the background image does not go outside of the designated area. I am only supporting 4.1 onwards so perhaps something has changed in core. Can someone confirm it still occurs for them in 4.1 or greater? If it does I really need replication steps. If a person can create a certificate from scratch that behaves wrongly remember each step and put it here. Thanks!

@mdjnelson mdjnelson removed this from In progress in 4.1.4, 4.2.5 Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants