Skip to content

Evaluation | Tom Winskell | Complete#223

Open
tomwinskell wants to merge 23 commits intoprojectshft:masterfrom
tomwinskell:main
Open

Evaluation | Tom Winskell | Complete#223
tomwinskell wants to merge 23 commits intoprojectshft:masterfrom
tomwinskell:main

Conversation

@tomwinskell
Copy link

Please read read me. Project uses modules and is bundled using WebPack. A live demo is available. The demo is rendered from the production build in the /dist folder on the gh-pages branch of my repository. The repository I am submitting a pull request for here is the source code for the bundled code.

…forecasted and current weather using a template string. Displays with weather icons. TODO: tidy up the renderWx file, potentially refactoring into additional files, smaller files and functions.
…t looks for loop placeholder, loops over the html filling values from array, returns new html string.
…st items to 12pm for the location chosen are returned.
…trings. Json also loaded by webpack rather than by fetch API.
const input = document.getElementById('search');
const dropdown = document.getElementById('dropdown');

function renderDropdown(filtered) {

Choose a reason for hiding this comment

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

argument name is not clear

.filter((city) => {
return city.name.toLowerCase().includes(query);
})
.slice(0, 10);

Choose a reason for hiding this comment

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

why the slice?

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

Successfully merging this pull request may close these issues.

2 participants