#224 i have removed a duplicate send some happiness now! button from th start page
Merged 4 years ago by jflory7. Opened 4 years ago by saidat.
fedora-commops/ saidat/fedora-happiness-packets correction1  into  master

@@ -48,9 +48,5 @@ 

      </p>

  

      {% include 'messaging/_message_list.html' %}

- 

-     <p class="text-center">

-         <a class="btn btn-primary btn-lg btn-responsive" href="{% url 'messaging:send' %}" role="button">Send some happiness now! <img class="emoji" src="static/images/emoji/loveletter.png" alt="💌" title="Love letter" aria-label="Emoji: Love letter"></a>

-     </p>

- 

+     

  {% endblock content %}

i have removed a second "send some happiness button now! " button from a start page since two button were doing th same thing

Hi @saidat, thanks for the PR. I verified the button is gone when I tested your changes. Before merging, I suggest two changes:

  • For the old button HTML, please remove it instead of commenting it out.

Reason: Since we use git, it's easy to look backwards later and "undo" a commit or change. It is better practice to delete lines instead of commenting them out if you are confident it isn't a temporary change.

  • Instead of deleting the bottom button, keep the top button and remove the lower button.

Reason: For our web application, sending a Happiness Packet is the most important activity. Since it is most important, I think it is better to keep it higher on the page and easier to catch the viewer's attention.

Can you please make these changes to this pull request? Thanks @saidat!

Metadata Update from @jflory7:
- Pull-request tagged with: PASSED, improvement, needs changes, type - frontend, type - summer coding
- Request assigned

4 years ago

1 new commit added

  • i have removed a duplicate button from the start html page
4 years ago

hello @jflory7 , if you get time can you please go through my pr, i have made changes as you told me . Thanks

Metadata Update from @jflory7:
- Pull-request untagged with: needs changes

4 years ago

Hi @saidat! This looks ready to go. Thanks for working on it. Merging! :clapper:

Pull-Request has been merged by jflory7

4 years ago

Hi @saidat! This looks ready to go. Thanks for working on it. Merging! 🎬

Your welcome @jflory7