Skip to content

Added location and schedule#81

Open
dulgheriubogdan wants to merge 2 commits into
devfrom
dulgheriu-bogdan-0
Open

Added location and schedule#81
dulgheriubogdan wants to merge 2 commits into
devfrom
dulgheriu-bogdan-0

Conversation

@dulgheriubogdan
Copy link
Copy Markdown
Collaborator

…petition

Comment thread client/src/components/Locatie.js Outdated
Comment on lines +19 to +21

//vrem vreme buna muie lu furtuna ca nu ne-a lasat sa punem animatia

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

irrelevant

Comment thread client/src/components/Locatie.js Outdated
</div>
<div className="flex-wrap rounded-lg pb-16 ">
<div className="relative w-[90%] md:w-4/6 w-4/5 rounded-lg mx-auto mt-16 border-4 border-[#78e800] shadow-inner ">
<img className="w-full rounded-lg" src={TestWebp} alt="Imagine" />
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

more suggestive alt value

Comment thread client/src/components/Program.js Outdated
</div>
<div className="flex-wrap rounded-lg pb-16 ">
<div className="relative w-[90%] md:w-4/6 w-4/5 rounded-lg mx-auto mt-16 border-4 border-[#78e800] shadow-inner ">
<img className="w-full rounded-lg" src={TestWebp} alt="Imagine" />
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

same here

Comment on lines +19 to +21

//vrem vreme buna muie lu furtuna ca nu ne-a lasat sa punem animatia

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

same here

Comment thread client/src/components/Program.js Outdated
@@ -0,0 +1,22 @@
import TestWebp from "../assets/img/webp/Indrumar/program.webp";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

change this name with : ProgramPicture or something

Comment thread client/src/components/Locatie.js Outdated
@@ -0,0 +1,22 @@
import TestWebp from "../assets/img/webp/Indrumar/harta.webp";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

change this name with LocationPicture or something

Comment thread client/src/views/Home.js Outdated
Comment on lines +17 to +18
import Locatie from "../components/Locatie";
import Program from "../components/Program";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

if the components name were in English, why did you name these in Romanian?

Comment thread client/package.json
@@ -43,6 +43,7 @@
]
},
"devDependencies": {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

where do you use this?

…tion to the images, changed from romanian to english names
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