Bug 16062

Summary: Remove CSS and images related to old "approve" and "deny" button styles
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: brendan, jonathan.druart, testopia, veron
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 16061, 16064    
Bug Blocks:    
Attachments: Bug 16062 - Remove CSS and images related to old "approve" and "deny" button styles
Bug 16062 - Remove CSS and images related to old "approve" and "deny" button styles
Bug 16062 - Remove CSS and images related to old "approve" and "deny" button styles
Bug 16062 - (follow-up) Remove CSS and images related to old "approve" and "deny" button styles

Description Owen Leonard 2016-03-14 13:51:06 UTC
Bug 15785 - Use Font Awesome icons in confirmation dialogs - Has made some CSS and images obsolete:

.dialog input.approve, td input.approve {
  background : #FFF url(../../img/approve.gif) no-repeat 4px center;
  padding : .4em .4em .4em 25px;  
}
 
.dialog input.deny {
   background : #FFF url(../../img/deny.gif) no-repeat 4px center;
   padding : .4em .4em .4em 25px;
 }
Comment 1 Owen Leonard 2016-03-14 14:27:55 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2016-03-14 17:09:21 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2016-03-16 11:39:51 UTC
What about
  1126 td input.approve {
  1127     background-color : #FFC;
  1128 }  
?
Comment 4 Owen Leonard 2016-03-16 12:30:06 UTC
(In reply to Jonathan Druart from comment #3)
> What about
>   1126 td input.approve {
>   1127     background-color : #FFC;
>   1128 }  
> ?

I didn't notice that. That was made obsolete by the overhaul of the lists interface.
Comment 5 Jonathan Druart 2016-03-17 09:50:20 UTC
Created attachment 49251 [details] [review]
Bug 16062 - Remove CSS and images related to old "approve" and "deny" button styles

Now that confirmation dialogs have all been udpated to use Font Awesome
icons, the old CSS for styling "approve" and "deny" buttons can be
removed along with the associated images.

To test, apply the patchs for Bugs 16061 and 16064 if necessary.

- Search the templates for instances of the "approve" or "deny" class
  which don't have a Font Awesome icon associated with it. There should
  be none.
- Confirm that there are no other references to the files "approve.gif"
  and "deny.gif"

Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 6 Jonathan Druart 2016-03-17 09:50:27 UTC
Created attachment 49252 [details] [review]
Bug 16062 - (follow-up) Remove CSS and images related to old "approve" and "deny" button styles

Remove another obsolete CSS rule for input.approve

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 7 Brendan Gallagher 2016-03-23 20:48:23 UTC
Pushed to Master - should be in the May 2016 release.  Thanks!