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

feat(browser): Give video and button tags an id attribute #138

Closed
wants to merge 1 commit into from

Conversation

cclaim
Copy link

@cclaim cclaim commented Nov 17, 2015

Allows developer to style both video and button tags in Browser.

@cclaim cclaim changed the title [Browser] Give video and button tag an id [Browser] Give video and button tags an id Nov 17, 2015
@riknoll
Copy link
Contributor

riknoll commented Nov 17, 2015

Does this have a corresponding JIRA issue? Also, I don't see your name in the ICLA list; could you please submit an ICLA?

@riknoll
Copy link
Contributor

riknoll commented Dec 1, 2015

@Clementp57 any updates?

@janpio janpio added this to ⏳ Waiting for Review in Apache Cordova: Plugin Pull Requests Sep 16, 2018
@janpio janpio moved this from ⏳ Waiting for Review to 🐣 New PR / Untriaged in Apache Cordova: Plugin Pull Requests Sep 16, 2018
@janpio
Copy link
Member

janpio commented Sep 16, 2018

Hey @Clementp57 , there seems to be a merge conflict now. Could you maybe fix this? Thanks.

@janpio janpio moved this from 🐣 New PR / Untriaged to 💥 Blocked: Conflict in Apache Cordova: Plugin Pull Requests Sep 16, 2018
@janpio janpio changed the title [Browser] Give video and button tags an id feat(browser): Give video and button tags an id attribute Jul 3, 2019
Copy link
Member

@janpio janpio left a comment

Choose a reason for hiding this comment

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

Feature sounds good, but we need to probably apply this in a new PR as this has a hard merge conflict now.

Apache Cordova: Plugin Pull Requests automation moved this from 💥 Blocked: Conflict to 🙅 Pending Approval Jul 3, 2019
@jcesarmobile
Copy link
Member

closing since the conflicts have not been fixed after a year

Apache Cordova: Plugin Pull Requests automation moved this from 🙅 Reviewed, but not approved to ☠️ Closed/Abandoned Jun 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants