Skip to content

[4.0] Fix edit account URI fragment#32067

Merged
HLeithner merged 1 commit intojoomla:4.0-devfrom
C-Lodder:patch-1
Jan 17, 2021
Merged

[4.0] Fix edit account URI fragment#32067
HLeithner merged 1 commit intojoomla:4.0-devfrom
C-Lodder:patch-1

Conversation

@C-Lodder
Copy link
Copy Markdown
Member

@C-Lodder C-Lodder commented Jan 17, 2021

Summary of Changes

Fix the URI fragment for the Edit Account so that it always opens the first tab

Testing Instructions

  1. Open the User Menu (top right)
  2. Click Edit Account
  3. Navigate to any tab...e.g Joomla API Token
  4. Click Close in the toolbar
  5. Re-do steps 1 and 2

Actual result BEFORE applying this Pull Request

The tab which was opened as last before the last closing is shown, e.g. Joomla API Token view

Expected result AFTER applying this Pull Request

My Profile Details view is shown

@richard67
Copy link
Copy Markdown
Member

richard67 commented Jan 17, 2021

I've tested with success, but the issue tracker doesn't show the PR yet, so I can't mark my test result.

There's a small mistake in the "Actual result BEFORE applying this Pull Request" section. It says "Joomla API Token view is shown", but it should be something like "The tab which was opened as last before the last closing is shown, e.g. Joomla API Token view.".

@C-Lodder
Copy link
Copy Markdown
Member Author

@richard67
Copy link
Copy Markdown
Member

Yes, meanwhile it appears in the tracker, but it hasn't noticed yet that it's a PR and so there is no test button. Will wait a while.

@richard67
Copy link
Copy Markdown
Member

The tracker shows also the wrong build (staging instead of 4.0-dev).

@HLeithner HLeithner closed this Jan 17, 2021
@HLeithner HLeithner reopened this Jan 17, 2021
@HLeithner HLeithner merged commit 0f955c8 into joomla:4.0-dev Jan 17, 2021
@HLeithner
Copy link
Copy Markdown
Member

Thanks

@HLeithner HLeithner added this to the Joomla 4.0 milestone Jan 17, 2021
dgrammatiko pushed a commit to dgrammatiko/joomla-cms that referenced this pull request Jan 17, 2021
dgrammatiko pushed a commit to dgrammatiko/joomla-cms that referenced this pull request Jan 20, 2021
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.

4 participants