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

Deprecate 'native' in favor of montage/ui/base #19

Open
3 tasks
NV opened this issue Aug 16, 2013 · 8 comments
Open
3 tasks

Deprecate 'native' in favor of montage/ui/base #19

NV opened this issue Aug 16, 2013 · 8 comments

Comments

@NV
Copy link
Collaborator

NV commented Aug 16, 2013

Refactor all components to use montage/ui/base where possible instead of montagejs/native. Digit uses montage/ui/base, make matte more consistent with digit.

  • Button
    • enabled property in serialization doesn’t change anything
    • 'hold' event is missing
@NV
Copy link
Collaborator Author

NV commented Aug 16, 2013

Working on it. I would assign it to myself but I don’t have permissions.
Update: thanks, have permissions now.

@ghost ghost assigned NV Aug 16, 2013
@NV
Copy link
Collaborator Author

NV commented Aug 16, 2013

After NV/matte@38a1a39 few specs started to fail. One of them related to "hold" event. I looked at abstract-button.js and press-composer.js and couldn’t find anything about "hold" event. Was it replaced by something else?

@francoisfrisch
Copy link
Contributor

I think we missed it when we refactored the code… Add it to the abstract class

On Aug 16, 2013, at 6:25 PM, Nikita Vasilyev [email protected] wrote:

After NV/matte@38a1a39 few specs started to fail. One of them related to "hold" event. I looked at abstract-button.js and press-composer.js and couldn’t find anything about "hold" event. Was it replaced by something else?


Reply to this email directly or view it on GitHub.

@NV
Copy link
Collaborator Author

NV commented Aug 19, 2013

When 'longAction' event suppose to fire for button? Should it fire on click and hold (for holdThreshold ms)? It currently doesn’t.

@aadsm
Copy link
Collaborator

aadsm commented Aug 19, 2013

It should be fired when the translate composer fires the longPress event

@NV
Copy link
Collaborator Author

NV commented Aug 19, 2013

You mean press composer? Abstract button doesn’t use translate composer.

I asked because 'longAction' doesn’t get triggered on mouse events and NV@6e17168 fails.

@francoisfrisch
Copy link
Contributor

So we should have a longAction when the longPress is fired.

longAction should get triggered with mouse events if is doesn't it's a bug.

@aadsm
Copy link
Collaborator

aadsm commented Aug 19, 2013

Yes, sorry, I did mean press composer.

@NV NV removed their assignment May 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants