Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

mateacademy-ai-mentor

mateacademy-ai-mentor created a review on a pull request on mate-academy/js_stateful-clones
Sorry, the pull request review was unsuccessful for the following reasons: > [!WARNING] > **No changes were made in the pull request since the last review.** Please make sure that you have added ...

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/py-mutable-immutable
The `collection_of_coins` variable is categorized as mutable, but sets in Python are mutable. Consider reviewing the categorization if the task requires strict separation between mutable and immuta...

View on GitHub

mateacademy-ai-mentor created a review on a pull request on mate-academy/py-mutable-immutable
Great job on implementing the task requirements! 🎉 Your code correctly categorizes the variables into mutable and immutable types as specified. Although there was a minor note regarding the `collec...

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/js_advanced_calculator
The `operate` method should update the `result` property by calling the callback with the current `result` and the `number` as arguments. Also, ensure that the callback is a function before invokin...

View on GitHub

mateacademy-ai-mentor created a review on a pull request on mate-academy/js_advanced_calculator
### Review Summary Great effort on implementing the `makeCalculator` function! 🎉 While the function is mostly well-implemented, there are a few critical issues that need attention to ensure the ca...

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/js_advanced_calculator
The `operate` method should not assume that the callback is one of the calculator's own methods. It should allow any function to be passed as a callback. Consider checking if `callback` is a functi...

View on GitHub

mateacademy-ai-mentor created a review on a pull request on mate-academy/js_advanced_calculator
**** Hello! 😊 It looks like you've done a great job implementing most of the calculator functionality. However, there's a critical issue with the `operate` method that needs to be addressed before...

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_search-bar-airbnb
Consider adding fallback fonts to ensure better compatibility across different systems. For example, you can use `font-family: Avenir, Helvetica, Arial, sans-serif;`.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_search-bar-airbnb
Avoid using the universal selector `*` for `box-sizing: border-box;` as it can impact performance. Consider applying it more selectively to specific elements or classes.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_search-bar-airbnb
The task requires using the `background-image` CSS property for the icon instead of an `img` tag. Ensure that the icon is implemented using CSS as specified in the task requirements.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_search-bar-airbnb
The `input` element in the big search form is missing the `data-qa="keypress"` attribute. Please add this attribute to meet the task requirements.

View on GitHub

mateacademy-ai-mentor created a review on a pull request on mate-academy/layout_search-bar-airbnb
Great job on addressing most of the issues from the previous review! 🎉 Your HTML and CSS files are well-structured and organized. However, there are a few critical points to keep in mind for future...

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variables `$button-color` and `$background-color-special-white` are used but not defined. Please define these variables to ensure the hover styles are applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$background-color-special-white` is used but not defined. Please define this variable to ensure the text color is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$button-color` is used but not defined. Please define this variable to ensure the background color is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$other-text-color` is used but not defined. Please define this variable to ensure the text color is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$margin-inline` is used but not defined. Please define this variable to ensure the margin is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$main-text-color` is used but not defined. Please define this variable to ensure the text color is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The mixin `on-screen4` is used but not defined. Please define this mixin to ensure the responsive design for four columns is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The mixin `on-screen3` is used but not defined. Please define this mixin to ensure the responsive design for three columns is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The `overflow-x: auto` on the navigation bar might cause unwanted scrollbars. Please verify if this behavior is intended.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The mixin `on-screen2` is used but not defined. Please define this mixin to ensure the responsive design for two columns is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$button-color` is used but not defined. Please define this variable to ensure the active link underline color is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$background-color-special-white` is used but not defined. Please define this variable to ensure the background color is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$button-color` is used but not defined. Please define this variable to ensure the hover color is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$margin-inline` is used but not defined. Please define this variable to ensure the margin is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$other-text-color` is used but not defined. Please define this variable to ensure the text color is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The variable `$background-color-special-white` is used but not defined. Please define this variable to ensure the background color is applied correctly.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The `src` attribute for the card image should be checked to ensure it points to the correct file relative to the HTML file location. Verify the path is correct.

View on GitHub

mateacademy-ai-mentor created a review comment on a pull request on mate-academy/layout_catalog
The `src` attribute for the logo image should be checked to ensure it points to the correct file relative to the HTML file location. Verify the path is correct.

View on GitHub

Load more