-
Notifications
You must be signed in to change notification settings - Fork 37
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
Redesign examples/*html
#65
Comments
I wish to work on this issue. |
Sure, @ShivankSahai, go ahead! |
Hi, I made changes to the index.html and wysiwyg.html files, and opened up a pull request, so please see if I did everything correctly. I wasn't able to understand the issue fully, so please tell me if I should have done anything else, or something more, and I'll work on it. |
@ShivankSahai Can you link your PR back here by adding a "Fixes #65" in its description (at the start)? I'll have a look. Thanks! |
Okay I skimmed through it and just wanted to confirm, did you test it out? Please make sure the inline editor works on all the examples in the index.html, also can you modify the css a bit (refer issue description above) Thanks! |
Are you still working on this? |
Pinging @ShivankSahai. @rain-light I guess you can go ahead with this. Let us know if you face any trouble along the way! |
Thank you, I started working on this. Anyways, npm found some high priority vulnerabilities and I run a "npm audit fix" to see what it could manage to fix alone. It then said that 2 of them would break something and I didn't continue. Maybe you already know, but just in case I reported this here |
Alright, I went exploring around and now I have some questions.
EDIT:
Thank you for the opportunity! |
Hi @rain-light! I appreciate your interest, and have tried to answer your questions below but feel free to ping me if you have any doubts whatsoever. Thanks!
This is actually a satellite library, and is used over at our
How about shifting the B3 code to B4 in a separate PR? Or maybe starting from scratch using B4?
Hmm, interesting. I'd be really excited to see some SCSS getting merged into this lib! Also, if you think this requires a complement revamp, go ahead!
It's actually the HTML blocks that are treated as examples for the demo to operate upon. Feel free to include it in your revamp if you feel like it! I have never used grunt, do you have some automated live server functionality such as with Gulp and browserSync? inline-markdown-editor/Gruntfile.js Line 60 in 4b8ccf4
Sure! You can enable that by simply typing in Oh and the icons! I could use a svg main file generated by icomoon.io (there's also FontAwesome if you want it) for better practice. Please do! |
Hello, I tried getting to work at this more than once this week, but unfortunately since I have to prepare Linear Algebra exam for the 1st of July and Java for the 10th I wouldn't have enough time to give this the amount of work it would need. If someone else wants to do this, feel free to do it. If no one will have taken it when I'll have finished, I'll join again. |
@rexagod If no one is working on this so can I do this? |
Sure, @NitinBhasneia! |
ok, thanks, @rexagod. I have seen that someone was already working on this so anything you want to add then, please. |
@rexagod I'd like to try this! |
Hi, this is a first-timers-only issue. This means we've worked to make it more legible to folks who either haven't contributed to our codebase before, or even folks who haven't contributed to open source before.
If that's you, we're interested in helping you take the first step and can answer questions and help you out as you do. Note that we're especially interested in contributions from people from groups underrepresented in free and open source software!
We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝
If you have contributed before, consider leaving this one for someone new, and looking through our general help wanted issues. Thanks!
🤔 What you will need to know.
Nothing. This issue is meant to welcome you to Open Source :) We are happy to walk you through the process.
📋 Step by Step
🙋 Claim this issue: Comment below. If someone else has claimed it, ask if they've opened a pull request already and if they're stuck -- maybe you can help them solve a problem or move it along!
📝 Update: To complete this PR, you need to redesign the
index.html
andselect.html
files in order to provide a more engaging and vivid demo to the new comers. What you implement mostly depends on your own acumen, with the essential points that,💾 Commit your changes
🔀 Start a Pull Request. There are two ways how you can start a pull request:
If you are familiar with the terminal or would like to learn it, here is a great tutorial on how to send a pull request using the terminal.
You can also edit files directly in your browser and open a pull request from there.
Please keep us updated
💬⏰ - We encourage contributors to be respectful to the community and provide an update within a week of claiming a first-timers-only issue. We're happy to keep it assigned to you as long as you need if you update us with a request for more time or help, but if we don't see any activity a week after you claim it we may reassign it to give someone else a chance. Thank you in advance!
If this happens to you, don't sweat it! Grab another open issue.
Is someone else already working on this?
🔗- We encourage contributors to link to the original issue in their pull request so all users can easily see if someone's already started on it.
👥- If someone seems stuck, offer them some help! Otherwise, take a look at some other issues you can help with. Thanks!
🤔❓ Questions?
Leave a comment below!
The text was updated successfully, but these errors were encountered: