Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent page jumping to top when clicking on 'request new password' link... #21

Merged
merged 1 commit into from
Oct 31, 2013

Conversation

JustCarmen
Copy link
Contributor

Hi Greg,

I think this one is clear. A little change to prevent the page jumping to top when clicking on the 'request new password' link. This happens mostly on the login block on the homepage because the screen is larger and it is more likely that someone has scrolled down a bit before clicking on the link. But I also changed the same function in the login form.

fisharebest added a commit that referenced this pull request Oct 31, 2013
Prevent page jumping to top when clicking on 'request new password' link...
@fisharebest fisharebest merged commit b1457a6 into fisharebest:master Oct 31, 2013
@fisharebest
Copy link
Owner

Another solution is to replace

<a href="#" id="xxx">click me</a>

with

<span id="xxx">click me</span>

@JustCarmen
Copy link
Contributor Author

I think that one is even better. Shall I change it?

@mnoorenberghe
Copy link
Contributor

No, that would be wrong semantically. Links should be implemented as <a> elements or you lose a lot of browser behaviour.

@JustCarmen JustCarmen deleted the wt-dev branch October 31, 2013 21:43
@fisharebest
Copy link
Owner

It is not really a link. We are not navigating anywhere. Semantically, would

<button></button>

be better?

The HTML5 spec says that href is optional. http://dev.w3.org/html5/markup/a.html#placeholder-hyperlink

<a id="xxx">click me</a>

I have not seen this before.

@mnoorenberghe
Copy link
Contributor

True, I forgot this was on one page now. I guess I still prefer <a> since it inherits the style for links and having more than one button below the login form would be somewhat confusing as people may think the added button would require a username to be typed in the <input>.

@fisharebest
Copy link
Owner

If we switch to bootstrap as our CSS framework, we could use something like

<button class="btn btn-link"></button>

bxbroze pushed a commit to bxbroze/webtrees that referenced this pull request Jan 16, 2016
@kexuebin kexuebin mentioned this pull request May 20, 2019
@ricklach ricklach mentioned this pull request Jul 11, 2019
@makitso makitso mentioned this pull request Aug 11, 2019
This was referenced Aug 25, 2019
@ghost ghost mentioned this pull request Aug 9, 2020
@webtrees-pesz webtrees-pesz mentioned this pull request Aug 18, 2023
@makitso makitso mentioned this pull request Oct 21, 2023
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.

None yet

3 participants