David Tsiklauri
David Tsiklauri
@AntonKhorev Currently, as I found in the project, for icons we use `.icon` class, which uses sprite with `background-image` and `background-position` properties. Adding icon to the sprite was solely for...
Thank you for the review @tomhughes I've updated accordingly @AntonKhorev If it's okay to directly use SVG instead of the `.icon` class practice of the project, I am all for...
@AntonKhorev Okay, thanks. I'll update it to ``. It will be more comfortable to reuse and will produce cleaner code.
PR was updated to include SVG in the HTML. I tried several different methods to move SVG from sprite, including adding it to the SvgHelper module and adding separate file...
This PR was updated according to the suggestion of @AntonKhorev. As described in the `CONTRIBUTING.md` only updated language config was `en`: > ## i18n > > If you make a...
Is there anything that should be fixed in this PR?
PR was updated. `master` was rebased as a base for this PR. Conflicts were resolved. There were some changes related to the header in the `master`, so logic and HTML...
PR was updated. `master` was rebased as a base for this PR. @tomhughes Code was changed according to comments.
Thanks for rerunning tests. PR was updated. `master` was merged to avoid previous test errors.
PR was updated. `master` was merged. @AntonKhorev Thank you for your PR. `Locale selector` was changed accordingly.