Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Outreachy Task Submission]: Add Delete Icon [Survey] #980

Closed
wants to merge 1 commit into from
Closed

[Outreachy Task Submission]: Add Delete Icon [Survey] #980

wants to merge 1 commit into from

Conversation

Cecil-o
Copy link

@Cecil-o Cecil-o commented Mar 28, 2024 •
edited
Loading

Added Delete Icon to Survey, this solve issue #4894

-These allow user to delete a survey when a single action is require for ease and fast navigation.

-I used font awesome icon in carrying out this task.
survey font
survey

@Cecil-o
included an icon library in index.html to use the delete icon under s…
b860329 
…urvey component, called the method and align the icon at the end in css
@Cecil-o Cecil-o changed the title included an icon library in index.html to use the delete icon under s… [Outreachy Task Submission]: Included an icon library in index.html to use the delete icon under s… Mar 28, 2024
@Cecil-o Cecil-o closed this Mar 28, 2024
@Cecil-o Cecil-o reopened this Mar 30, 2024
@Cecil-o Cecil-o changed the title [Outreachy Task Submission]: Included an icon library in index.html to use the delete icon under s… [Outreachy Task Submission]: Add Delete Icon [Survey] Mar 30, 2024
tuxpiper
tuxpiper requested changes Apr 2, 2024
View reviewed changes
Copy link
Member

@tuxpiper tuxpiper left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution!

Could you please add some more details to the issue that you created (ushahidi/platform#4894)? It's difficult to follow and make sure I understand what you are intending to fix here.

apps/web-mzima-client/src/env.json
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this change, it's not useful to others

@tuxpiper
Copy link
Member

tuxpiper commented Apr 2, 2024

Thank you for the contribution!

Could you please add some more details to the issue that you created (ushahidi/platform#4894)? It's difficult to follow and make sure I understand what you are intending to fix here.

A screenshot would be helpful

@Cecil-o
Copy link
Author

Cecil-o commented Apr 11, 2024

Thank you for the contribution!

Could you please add some more details to the issue that you created (ushahidi/platform#4894)? It's difficult to follow and make sure I understand what you are intending to fix here.

Hello, my apology for the delay, had some technical issue. The changes made should be clear now. Thanks

@Angamanga Angamanga closed this Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tuxpiper tuxpiper tuxpiper requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Cecil-o @tuxpiper @Angamanga