Closed soustab10 closed 5 months ago
I would like to work on it @TimidRobot
Hey @nitin-pandita , since you are working on #712 ,shall I take up this issue?
Hey @nitin-pandita , since you are working on #712 ,shall I take up this issue?
sure , go for it
@TimidRobot the fix to this issue is in the creativecommons/vocabulary-styles
repository in the following file footer.scss
It needs to be updated and the following code added in the mobile()
section:
@include mobile() {
.subscription {
.newsletter {
width: 100%;
}
}
}
Should I submit a PR directly in that repository or create an issue there?
@adidevs Hmmm... I'll let @possumbilities weigh in. I suspect we may want to mark this issue as blocked pending changes to Vocabulary.
@adidevs I would concur with @TimidRobot. There's a fix brewing that would address this differently, but is precisely this Issue in the new changes to Vocabulary. I'd say best to make this status: blocked
for now.
Thank you for your feedback @possumbilities and @TimidRobot !🙌
as i see there are no PR for this i am gonna genrate one
see u
Hey @soustab10 can i work on the above issue?
I want to work on this issue please assign me this issue.
Can I work on this issue.
I want to work on this issue please assign me this issue.
Hey I clearly understood what issue is please assign this issue to me, I will sole it really quickly.
@Ritesh92A @Shailendrasingh189 @abhishekjain58
Please see Contribution Guidelines — Creative Commons Open Source for how we manage issues and PRs (we generally don't assign issues prior to resolution).
Hi, I created a PR with a fix for this: https://github.com/creativecommons/creativecommons.github.io-source/pull/763
I've tested it locally, please review @TimidRobot
Description
The email text box is not optimized for mobile view.
Reproduction
Expectation
The width of the email text box should be adjusted for the device width.
Screenshots
Additional context
Resolution