Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

✍️ Redesign: New User-Interface #4

Closed
N3v1 opened this issue Aug 2, 2023 · 4 comments
Closed

✍️ Redesign: New User-Interface #4

N3v1 opened this issue Aug 2, 2023 · 4 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed question Further information is requested ToDo

Comments

@N3v1
Copy link
Owner

N3v1 commented Aug 2, 2023

Hello everyone 👋,
it would help me if you would help me with the redesign of the calculator.

What do you need to be able to do to help here?:

  • Basic HTML knowledge
  • Basic/Advanced CSS knowledge

⚠️ Important Note: Please sync your forks to prevent merge conflicts ⚠️

@N3v1 N3v1 added documentation Improvements or additions to documentation enhancement New feature or request question Further information is requested ToDo help wanted Extra attention is needed labels Aug 2, 2023
@N3v1 N3v1 pinned this issue Aug 2, 2023
@N3v1 N3v1 added the good first issue Good for newcomers label Aug 2, 2023
anubhav1206 added a commit to anubhav1206/Calculator that referenced this issue Aug 3, 2023
@N3v1
Copy link
Owner Author

N3v1 commented Aug 3, 2023

Hi @anubhav1206 👋,
thank you for your help, but I get an error message in the style.css file: 'Also define the standard property "background-clip" for compatibility reasons css(vendorPrefix).

Determines the background painting area.

(Chrome, Safari 3)'

The fault may also be on my side 🙁. Maybe I copied your changes wrong. If you still have time and desire, could you please try to fix the error (if the error message is also visible to you)? Otherwise, I would merge your PR and create a new bug in the Issue tab.
Thanks in advance 🙏 - NH

@N3v1
Copy link
Owner Author

N3v1 commented Aug 3, 2023

Heres a screenshot of the error:
Bildschirmfoto 2023-08-03 um 11 21 27

If you have questions about the translation of the error message, feel free to ask me 🙂

@anubhav1206
Copy link
Contributor

Heres a screenshot of the error: Bildschirmfoto 2023-08-03 um 11 21 27

If you have questions about the translation of the error message, feel free to ask me 🙂

Hi @N3v1 , that seems to be a warning rather than an error. Trying to remove the warning breaks the code on my side, I would suggest just leaving it as it is.

@N3v1
Copy link
Owner Author

N3v1 commented Aug 3, 2023

Hi @anubhav1206, 👋
i will now merge your PR. Thanks for your help. I'm sorry that I may have caused you any inconvenience. I'm still new to web development. Thank you very much - NH

N3v1 added a commit that referenced this issue Aug 3, 2023
@N3v1 N3v1 closed this as completed Aug 3, 2023
@N3v1 N3v1 unpinned this issue Aug 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed question Further information is requested ToDo
Projects
None yet
Development

No branches or pull requests

2 participants