Bug 21151 removed [% INCLUDE 'doc-head-close.inc' %] from sru_modmapping.tt. It means the page loads without any CSS at all. I don't think this was intentional?
Created attachment 77982 [details] [review] Bug 21243: Regression: SRU mapping popup for bibliographic records is unstyled This patch corrects a regression caused by Bug 21151: The include file which adds CSS to the page was removed from the template. To test, apply the patch and go to Administration -> Z39.50/SRU servers -> New SRU server. - Click the "Modify" button next to "SRU Search fields mapping" - Verify that the pop-up window is correctly styled.
Hm, my patch on bug 21552 didn't remove it, only added a parameter - wonder what happened there? +[% INCLUDE 'doc-head-close.inc' popup => 1 %]
Created attachment 78026 [details] [review] Bug 21243: Regression: SRU mapping popup for bibliographic records is unstyled This patch corrects a regression caused by Bug 21151: The include file which adds CSS to the page was removed from the template. To test, apply the patch and go to Administration -> Z39.50/SRU servers -> New SRU server. - Click the "Modify" button next to "SRU Search fields mapping" - Verify that the pop-up window is correctly styled. Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>
Created attachment 78041 [details] [review] Bug 21243: Regression: SRU mapping popup for bibliographic records is unstyled This patch corrects a regression caused by Bug 21151: The include file which adds CSS to the page was removed from the template. To test, apply the patch and go to Administration -> Z39.50/SRU servers -> New SRU server. - Click the "Modify" button next to "SRU Search fields mapping" - Verify that the pop-up window is correctly styled. Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Is reverting bug 21151 and repushing it correctly also an option? It adds a missing div as well.
(In reply to Katrin Fischer from comment #5) > Is reverting bug 21151 and repushing it correctly also an option? It adds a > missing div as well. The div does seem to have been added, it was a typo during rebase that I deleted the line, I think we can just add it here bug 13618 would make revert more difficult I think
(In reply to Nick Clemens from comment #6) > (In reply to Katrin Fischer from comment #5) > > Is reverting bug 21151 and repushing it correctly also an option? It adds a > > missing div as well. > > The div does seem to have been added, it was a typo during rebase that I > deleted the line, I think we can just add it here > > bug 13618 would make revert more difficult I think Looking again, I think that extra closing div is incorrect. We have a closing div in intranet-bottom.inc - with 21151 we have an extra closing tag (run the generated html through the wc3 validator)
Created attachment 78044 [details] [review] Bug 21243: (follow-up) Remove extra '</div>' tag To test: 1 - Go to Admin->Z39/SRU servers 2 - Pick a server or add one 3 - Click 'Modify' button for mappings 4 - Copy the HTML of the popup into w3c validator https://validator.w3.org/#validate_by_input 5 - Note extra closing div 6 - Apply patch 7 - Repeat 8 - Success
Ok, probably also fixed by adding SRU for authorities then. I had checked the validator at the time, but the other patches altered page structure.
Awesome work all! Pushed to master for 18.11
Pushed to 18.05.x for 18.05.03
Depends on Bug 21151 not in 17.11.x