? NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar anmode
anmode
6 Mar 2022

Pull Request for #37109
Feature:
Put the Extra logout button outside the dropdown menu

Summary of Changes
I have just configured the administrator/modules/mod_user/tmpl/default.php file in the codebase.

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required
No changes required

avatar anmode anmode - open - 6 Mar 2022
avatar anmode anmode - change - 6 Mar 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Mar 2022
Category Modules Administration NPM Change Front End Plugins
avatar richard67
richard67 - comment - 6 Mar 2022

Have you noticed that there are other pull requests by other people for the same issue?

avatar anmode
anmode - comment - 6 Mar 2022

Have you noticed that there are other pull requests by other people for the same issue?

yes @richard67 but there were some issues in those PRs that is why I request a new PR. In one of the PR some checks failed and I guess one has base conflicts.

avatar brianteeman
brianteeman - comment - 6 Mar 2022

well this PR is no better :(

avatar anmode
anmode - comment - 6 Mar 2022

well this PR is no better :(

Why :( ? I will make necessary changes in it soon.

avatar richard67
richard67 - comment - 6 Mar 2022

Wouldn't it be more fair to give the authors of the other PRs the same chance to fix their PRs? And wouldn't it be more effective to focus on some other issue - we have enough of them - instead of one where already 2 other people are working on?

avatar anmode
anmode - comment - 6 Mar 2022

I have tested code on my localhost the extra log out button working fine! With same UI
If then too there js any error pls guide a little i will try to solve it

avatar anmode
anmode - comment - 6 Mar 2022

Wouldn't it be more fair to give the authors of the other PRs the same chance to fix their PRs? And wouldn't it be more effective to focus on some other issue - we have enough of them - instead of one where already 2 other people are working on?

Well yes actually i thought couple of days passed on those PR and feature not fixed that's why i tried and generated it.
So may i close this PR :(? Or let it open

avatar richard67
richard67 - comment - 6 Mar 2022

It's not a question of working or not. It has to be accessible, it has to remove the then redundant logout option from the drop down ... all that has been discussed in the other PRs, #37183 and #37165 . Maybe you should check the comments in these PRs first and also the comments in the issue #37109 (which us not really an issue or bug but a feature request or proposal for UX improvement) so you don't make the same mistakes again as the other PRs did or still do.

You also waste our time with that just by the way if we have to explain the same things again here and there.

avatar brianteeman
brianteeman - comment - 6 Mar 2022

well this PR is no better :(

Why :( ? I will make necessary changes in it soon.

  1. There are unrelated files in the pull request
  2. You have removed ALL formatting from the file. So not only does it break the style guidelines it is now impossible to see here on github what has changed as you have changed EVERY line
  3. Plus everything @richard67 said
avatar anmode
anmode - comment - 6 Mar 2022

It's not a question of working or not. It has to be accessible, it has to remove the then redundant logout option from the drop down ... all that has been discussed in the other PRs, #37183 and #37165 . Maybe you should check the comments in these PRs first and also the comments in the issue #37109 (which us not really an issue or bug but a feature request or proposal for UX improvement) so you don't make the same mistakes again as the other PRs did or still do.

You also waste our time with that just by the way if we have to explain the same things again here and there.

Sorry my bad actually i m new to open source.
I will not repeat my mistakes actually i haven't checked the comments well. I also misunderstood something sorry for that.
I will work upon other issue :) and solve it keeping in mind all things

avatar richard67
richard67 - comment - 6 Mar 2022

@anmode I suggest you close your PR in favour of #37183 , because that one seems to be closest to the right solution.

We appreciate that you have made your PR because it shows your interest on working on Joomla.

But it's not easy at the beginning when you start to collaborate with others on a huge software project.

You should not be disappointed or discouraged but try to learn from it.

There are many things you can do to help Joomla and get more familiar with using it and with the code. A good start is always to test other people's pull requests.

And sooner or later there will be an issue for which you can provide a fix and make a nice pull request.

avatar anmode
anmode - comment - 6 Mar 2022

well this PR is no better :(

Why :( ? I will make necessary changes in it soon.

  1. There are unrelated files in the pull request
  2. You have removed ALL formatting from the file. So not only does it break the style guidelines it is now impossible to see here on github what has changed as you have changed EVERY line
  3. Plus everything @richard67 said

Yes sorry for that i haven't know about it, i will surely not repeat my mistakes.

avatar richard67
richard67 - comment - 6 Mar 2022

Closing as stated above. Thanks for your contribution, and heads up. The next one will be better for sure.

avatar richard67 richard67 - change - 6 Mar 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-03-06 11:17:17
Closed_By richard67
Labels Added: ? NPM Resource Changed ?
avatar richard67 richard67 - close - 6 Mar 2022

Add a Comment

Login with GitHub to post a comment