'<div id="opacheader">' should be removed from masthead.inc because the same markup is generated by koha_news_block.
Created attachment 99709 [details] [review] Bug 24746: Duplicate id in opacheader markup This patch removes '<div id="opacheader">' from masthead.inc because the same markup is generated by koha_news_block. To reproduce the bug, make sure there is an opacheader news block defined. Add this to the OPACUserCSS system preference: View the main page of the OPAC. You should see two blue borders around the opacheader content. Apply the patch and reload the OPAC page. There should now be only one blue border.
Created attachment 99738 [details] [review] Bug 24746: Duplicate id in opacheader markup This patch removes '<div id="opacheader">' from masthead.inc because the same markup is generated by koha_news_block. To reproduce the bug, make sure there is an opacheader news block defined. Add this to the OPACUserCSS system preference: View the main page of the OPAC. You should see two blue borders around the opacheader content. Apply the patch and reload the OPAC page. There should now be only one blue border. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Tested whith OPACUserCSS -> #opacheader { border: 1px solid blue; padding: 1px; } don't know the intended value but works :) Work as described following test plan, no errors.
I wonder if this could create issues for people who have tied markup etc. to the id without having the pref filled?
Created attachment 99802 [details] [review] Bug 24746: Duplicate id in opacheader markup This patch removes '<div id="opacheader">' from masthead.inc because the same markup is generated by koha_news_block. To reproduce the bug, make sure there is an opacheader news block defined. Add this to the OPACUserCSS system preference: View the main page of the OPAC. You should see two blue borders around the opacheader content. Apply the patch and reload the OPAC page. There should now be only one blue border. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Tested whith OPACUserCSS -> #opacheader { border: 1px solid blue; padding: 1px; } don't know the intended value but works :) Work as described following test plan, no errors.
Created attachment 99803 [details] [review] Bug 24746: Duplicate id in opacheader markup This patch removes '<div id="opacheader">' from masthead.inc because the same markup is generated by koha_news_block. To reproduce the bug, make sure there is an opacheader news block defined. Add this to the OPACUserCSS system preference: View the main page of the OPAC. You should see two blue borders around the opacheader content. Apply the patch and reload the OPAC page. There should now be only one blue border. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Tested whith OPACUserCSS -> #opacheader { border: 1px solid blue; padding: 1px; } don't know the intended value but works :) Work as described following test plan, no errors.
Created attachment 99804 [details] [review] Bug 24746: Duplicate id in opacheader markup This patch removes '<div id="opacheader">' from masthead.inc because the same markup is generated by koha_news_block. To reproduce the bug, make sure there is an opacheader news block defined. Add this to the OPACUserCSS system preference: View the main page of the OPAC. You should see two blue borders around the opacheader content. Apply the patch and reload the OPAC page. There should now be only one blue border. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Tested whith OPACUserCSS -> #opacheader { border: 1px solid blue; padding: 1px; } don't know the intended value but works :) Work as described following test plan, no errors.
Created attachment 99805 [details] [review] Bug 24746: Duplicate id in opacheader markup This patch removes '<div id="opacheader">' from masthead.inc because the same markup is generated by koha_news_block. To reproduce the bug, make sure there is an opacheader news block defined. Add this to the OPACUserCSS system preference: View the main page of the OPAC. You should see two blue borders around the opacheader content. Apply the patch and reload the OPAC page. There should now be only one blue border. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Tested whith OPACUserCSS -> #opacheader { border: 1px solid blue; padding: 1px; } don't know the intended value but works :) Work as described following test plan, no errors. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Nice work everyone! Pushed to master for 20.05
not backported 19.11.x due to dependencies