Add Navtrail with dropdown

[omar2205]

This adds a dropdown menu and a user avatar

Not logged

Logged in

[omar2205]

big oof 😛

[omar2205]

@w3cj this error

Error:   10:23  error  Parse errors in imported module './NavTrail.svelte': '>' expected. (1:8)
import/no-named-as-default

is related to #37 issue

[w3cj]

I think you will need to merge with main for the CI checks to pass.

[omar2205]

I just have to do another commit for it to recheck it

[brilvio]

Your branch is 3 commits behind main, maybe rebase or merge main into your branch.

[w3cj]

Regarding the $app multiple import error, I think the fix in the tsconfig.json might have been wrong. Can you try removing the $app aliases in there and add "./node_modules/@sveltejs/kit/types/ambient.d.ts" to the includes array? I'm not near a PC to try right now.

[omar2205]

removing the tsconfig fix results in import/no-unresolved

[w3cj]

Please merge / rebase with master. I updated the eslint config to handle multiple $app imports.

[omar2205]

Still, we get import/no-named-as-default-member

[w3cj]

Looks like the rule import/no-named-as-default-member is the issue instead of import/no-named-as-default??

[omar2205]

Not sure if it's both or just one of them.

[omar2205]

Yeah, we need both!

     'import/no-named-as-default': 0,
     'import/no-named-as-default-member': 0,

[w3cj]

Let's disable these rules for .svelte files specifically. Do you mind opening another PR?

This section in the eslint config can be updated and we can remove 'import/no-named-as-default': 0, in the general rules:

	overrides: [
		{
			files: ['**/*.svelte'],
			parser: 'svelte-eslint-parser',
			parserOptions: {
				parser: '@typescript-eslint/parser',
			},
			rules: {
				'import/no-named-as-default': 0,
				'import/no-named-as-default-member': 0,
			},
		},
	],

[omar2205]

OK, will test it and open a PR if it worked!

[omar2205]

btw, I keep getting this warning

`parseForESLint` from parser `@typescript-eslint/parser` is invalid and will just be ignored