Skip to content

Commit

Permalink
Add qtip
Browse files Browse the repository at this point in the history
  • Loading branch information
mrcrankhank committed Oct 20, 2015
1 parent 7ac9f76 commit ba8ea1b
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 3 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ Take a look at the github releases for detailed information about the features.
- [x] Objects can be added multiple times
- [x] JavaScript problems in ietusers menu
- [x] Wrong success message when deleting a target + page is empty afterwards
- [ ] Forced delete of a target does not work if a "ALL" acl is configured
- [x] Forced delete of a target does not work if a "ALL" acl is configured
- [ ] add_iqn_to_file() function might remove comments

## Roadmap
Expand Down
3 changes: 2 additions & 1 deletion app/views/targets/delete_target.php
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
<div class="col-md-12">
<label>
<input id="force" type="checkbox" <?php if (isset($data['session'])) echo 'checked disabled' ?>> Force
</label> (Delete even if in use, requires 'Delete acl')
</label> <a href="#"><span id="icon_force" class="glyphicon glyphicon-question-sign glyphicon-20" aria-hidden="true"></span></a>
</div>
</div>
<div class="row top-buffer">
Expand All @@ -43,6 +43,7 @@
require(['pages/target/deletetarget', 'domReady'], function (methods, domReady) {
domReady(function () {
methods.add_event_handler_deletetargetbutton();
methods.add_qtip();
});
});
});
Expand Down
13 changes: 12 additions & 1 deletion public/js/pages/target/deletetarget.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
define(['jquery', 'mylibs', 'sweetalert'], function ($, mylibs, swal) {
define(['jquery', 'mylibs', 'sweetalert', 'qtip'], function ($, mylibs, swal, qtip) {
var methods;

return methods = {
Expand Down Expand Up @@ -63,6 +63,17 @@ define(['jquery', 'mylibs', 'sweetalert'], function ($, mylibs, swal) {
});
}
});
},
add_qtip: function() {
$('#icon_force').qtip({
content: {
text: 'Delete target even if in use. Requires \'Delete acl\'. ' +
'Does not work, if a \'ALL\' initiator acl is configured for this or all targets!'
},
style: {
classes: 'qtip-youtube'
}
});
}
};
});

0 comments on commit ba8ea1b

Please sign in to comment.