Skip to content

Issues: fack2/languagesSearch

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.

Sign up for GitHub

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

28 Open 4 Closed
28 Open 4 Closed
Author
Filter by author
Loading
Label
Filter by label
Loading
Use alt + click/return to exclude labels
or ⇧ + click/return for logical OR
Projects
Filter by project
Loading
Milestones
Filter by milestone
Loading
Assignee
Filter by who’s assigned
Sort
Sort by
Newest Oldest Most commented Least commented Recently updated Least recently updated Best match
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀

Issues list

Handling server errors
#42 opened Jul 11, 2019 by ameentalahmeh
1
not semantic to put a list in a div
#41 opened Jul 11, 2019 by eadehemingway
You don't have the website's wireframe picture in the readme file.
#40 opened Jul 11, 2019 by 95Rawan
1
the flow sketch ReadMe is not clear :: make it bigger
#39 opened Jul 11, 2019 by Muniralsharif
1
improve function name
#37 opened Jul 11, 2019 by eadehemingway
declare variables close to where they are used
#36 opened Jul 11, 2019 by eadehemingway
get rid of 'var's
#35 opened Jul 11, 2019 by eadehemingway
replace all function declarations with arrow functions
#34 opened Jul 11, 2019 by eadehemingway
It's better to use "textContent" instead of "innerHTML"
#33 opened Jul 11, 2019 by sulieman1
1
console.logs should never be merged into master!
#32 opened Jul 11, 2019 by eadehemingway
No need for arabic inisilaization
#31 opened Jul 11, 2019 by 95Rawan
1
'bu' is not a good id name!
#30 opened Jul 11, 2019 by eadehemingway
replace with filter
#29 opened Jul 11, 2019 by eadehemingway
make variable names explicit
#28 opened Jul 11, 2019 by eadehemingway
It's better to use meaningful variable names
#27 opened Jul 11, 2019 by sulieman1
1
why is this let?
#26 opened Jul 11, 2019 by eadehemingway
improve error handling!
#25 opened Jul 11, 2019 by eadehemingway
move website link to the description of repo
#24 opened Jul 11, 2019 by eadehemingway
Download pretier so your code looks nice
#23 opened Jul 11, 2019 by 95Rawan
1
you dont need './' in path.join
#22 opened Jul 11, 2019 by eadehemingway
no need to store this in a variable
#21 opened Jul 11, 2019 by eadehemingway
remove unnecessary console logs
#20 opened Jul 11, 2019 by sulieman1
1
the point of path.join is to divide the path
#19 opened Jul 11, 2019 by eadehemingway
Var is lazy, Try using let or const depending on the type of variable :)
#18 opened Jul 11, 2019 by sulieman1
1
It should be mobile first
#17 opened Jul 11, 2019 by 95Rawan
1
ProTip! Adding no:label will show everything without a label.