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

[8.0] web_widget_digital_signature #267

Merged
merged 21 commits into from
Feb 25, 2016
Merged

Conversation

eLBati
Copy link
Member

@eLBati eLBati commented Nov 26, 2015

Web Digital Signature

This module provides the functionality to store digital signature image for a record.

Usage

To use this module, you need to add widget="signature" to your binary field in your view.
You can specifify signature dimensions like the following:
<field name="signature_image" widget="signature" width="400" height="100"/>

Web Digital Signature for users

This module adds a signature field (using web_widget_digital_signature) to user form.

@eLBati
Copy link
Member Author

eLBati commented Nov 26, 2015

@JayVora-SerpentCS hi, I'm proposing this your module to be added to OCA/web. I splitted it in 2 modules, a bit improved, and added another one: web_widget_digital_signature_sale_order

@hbrunn hi, about your comment on the old PR #115 (comment), could you show an example of code the would fix it? Thanks

@hbrunn
Copy link
Member

hbrunn commented Nov 26, 2015

@eLBati in any case, the save function should end with

return this._super.apply(this, arguments)

Otherwise, you deactivate whatever other modules do in this function.

But as I wrote in the comment, I think you're better off not messing with the save function at all, but override get_value in instance.web.form.FieldSignature and do your click there. Even better would be a refactoring that you call a function that the click event handler also calls, then you avoid possible side effects of the clicking.

Here the relevant docstring: https://github.com/OCA/OCB/blob/8.0/addons/web/static/src/js/view_form.js#L2036

@@ -0,0 +1,22 @@
# -*- coding: utf-8 -*-

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@elicoidal
Copy link

Thanks for the contribution

@eLBati
Copy link
Member Author

eLBati commented Nov 27, 2015

@hbrunn you can see my refactoring test at https://github.com/eLBati/web/blob/add_web_digital_sign_8_ref/web_widget_digital_signature/static/src/js/digital_sign.js
but I did not manage to make it work (I am not very confident with web widgets).

At the moment, I can avoid breaking inheritance with 3639561

@eLBati
Copy link
Member Author

eLBati commented Nov 27, 2015

@elicoidal thanks, I made the changes

@eLBati
Copy link
Member Author

eLBati commented Dec 29, 2015

@JayVora-SerpentCS @pedrobaeza hi, we renamed the modules and moved sale_order_widget_digitized_signature to OCA/sale-workflow#252

@archetipo
Copy link
Member

👍

1 similar comment
@JayVora-SerpentCS
Copy link
Contributor

+1


=====================
Web Digitized Signature
=====================
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eLBati this == lines have to have same length as the text contained between them

@oihane
Copy link
Contributor

oihane commented Jan 18, 2016

Two things to fix in README files, but the rest LGTM 👍

@eLBati
Copy link
Member Author

eLBati commented Jan 19, 2016

@oihane fixed, thanks

@hbrunn
Copy link
Member

hbrunn commented Jan 19, 2016

@eLBati if you remove the company logo, I think this is fit for merging: https://github.com/eLBati/web/blob/add_web_digital_sign_8/web_widget_digitized_signature/static/src/img/icon.png

@hbrunn hbrunn added this to the 8.0 milestone Jan 19, 2016
* http://jqueryui.com
* Includes: core.css, accordion.css, autocomplete.css, button.css, datepicker.css, dialog.css, draggable.css, menu.css, progressbar.css, resizable.css, selectable.css, selectmenu.css, slider.css, sortable.css, spinner.css, tabs.css, tooltip.css, theme.css
* To view and modify this theme, visit http://jqueryui.com/themeroller/?ffDefault=segoe%20ui%2CArial%2Csans-serif&fwDefault=bold&fsDefault=1.1em&cornerRadius=6px&bgColorHeader=ece8da&bgTextureHeader=gloss_wave&bgImgOpacityHeader=100&borderColorHeader=d4ccb0&fcHeader=433f38&iconColorHeader=847e71&bgColorContent=f5f3e5&bgTextureContent=highlight_hard&bgImgOpacityContent=100&borderColorContent=dfd9c3&fcContent=312e25&iconColorContent=808080&bgColorDefault=459e00&bgTextureDefault=highlight_hard&bgImgOpacityDefault=15&borderColorDefault=327E04&fcDefault=ffffff&iconColorDefault=eeeeee&bgColorHover=67b021&bgTextureHover=highlight_soft&bgImgOpacityHover=25&borderColorHover=327E04&fcHover=ffffff&iconColorHover=ffffff&bgColorActive=fafaf4&bgTextureActive=highlight_hard&bgImgOpacityActive=100&borderColorActive=d4ccb0&fcActive=459e00&iconColorActive=8DC262&bgColorHighlight=fcf0ba&bgTextureHighlight=glass&bgImgOpacityHighlight=55&borderColorHighlight=e8e1b5&fcHighlight=363636&iconColorHighlight=8DC262&bgColorError=ffedad&bgTextureError=highlight_soft&bgImgOpacityError=95&borderColorError=e3a345&fcError=cd5c0a&iconColorError=cd0a0a&bgColorOverlay=2b2922&bgTextureOverlay=inset_soft&bgImgOpacityOverlay=15&opacityOverlay=90&bgColorShadow=cccccc&bgTextureShadow=highlight_hard&bgImgOpacityShadow=95&opacityShadow=20&thicknessShadow=12px&offsetTopShadow=-12px&offsetLeftShadow=-12px&cornerRadiusShadow=10px
* Copyright 2014 jQuery Foundation and other contributors; Licensed MIT */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't even see this file included, can it just go away?

@pedrobaeza
Copy link
Member

Please remove company logo in web_widget_digitized_signature/static/src/img/icon.png

@eLBati
Copy link
Member Author

eLBati commented Jan 26, 2016

@pedrobaeza @hbrunn removed company logo and not used CSS
Thanks

@hbrunn
Copy link
Member

hbrunn commented Jan 27, 2016

👍

@eLBati
Copy link
Member Author

eLBati commented Feb 25, 2016

I think this can be merged

hbrunn added a commit that referenced this pull request Feb 25, 2016
[8.0] web_widget_digital_signature
@hbrunn hbrunn merged commit 3c67964 into OCA:8.0 Feb 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants