Merge pull request #19636 from guardian/sa-remove-sign-in-link · devhttps/frontend@a44c3bc · GitHub

devhttps / frontend Public

forked from guardian/frontend

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Wiki
• Security and quality
• Insights

Additional navigation options

Commit a44c3bc

Browse filesBrowse the repository at this point in the historyBrowse files

SiAdcock

authored

Merge pull request guardian#19636 from guardian/sa-remove-sign-in-link

Remove sign in link if user is signed in

2 parents e4ae03f + ce7cac0 commit a44c3bc
Copy full SHA for a44c3bc

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• static/src/javascripts/projects/common/modules/navigation
  • user-account.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎static/src/javascripts/projects/common/modules/navigation/user-account.js‎

Copy file name to clipboardExpand all lines: static/src/javascripts/projects/common/modules/navigation/user-account.js

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -52,7 +52,7 @@ const showMyAccountIfNecessary = (): void => {
52	52	return fastdom
53	53	.write(() => {
54	54	signIns.forEach(signIn => {
55		- signIn.classList.add('u-h');
	55	+ signIn.remove();
56	56	});
57	57
58	58	accountActionsLists.forEach(accountActions => {

0 commit comments

Comments

0 (0)