Project

General

Profile

Actions

Defect #23777

closed

Login button is too thin on WebKit browsers

Added by Go MAEDA over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Category:
UI
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

Affected version: trunk (r15595 and later)

Firefox:
No problem.

Chrome:
padding property of input#login-submit seems to be ineffective.


Files

login-firefox.png (13.9 KB) login-firefox.png Go MAEDA, 2016-09-07 03:42
login-chrome.png (14 KB) login-chrome.png Go MAEDA, 2016-09-07 03:42
fix-login-button-is-too-thin.diff (1.4 KB) fix-login-button-is-too-thin.diff patch Go MAEDA, 2016-09-07 04:20
fix-login-button-is-too-thin_v2.diff (566 Bytes) fix-login-button-is-too-thin_v2.diff Marius BĂLTEANU, 2016-10-01 14:38
Actions #1

Updated by Go MAEDA over 7 years ago

This is a patch to fix this issue: fix-login-button-is-too-thin.diff
Changes input[type="submit"] to button[type="submit"].

Actions #2

Updated by martin Jacob over 7 years ago

  • File Plombier Créteil.jpg added
Actions #3

Updated by Go MAEDA over 7 years ago

  • File deleted (Plombier Créteil.jpg)
Actions #4

Updated by Toshi MARUYAMA over 7 years ago

I cannot see any problem on Windows 10 Chrome 53.0.2785.101 m (64-bit).

Actions #5

Updated by Go MAEDA over 7 years ago

Toshi MARUYAMA wrote:

I cannot see any problem on Windows 10 Chrome 53.0.2785.101 m (64-bit).

Thank you for checking on Windows. I tested on Windows after seeing your post and found that the problem can not be reproduced on Windows.

I reproduced this problem on:

  • Mac OS X 10.11.6 - Chrome 52.0.2743.116 (64-bit)
  • Mac OS X 10.11.6 - Safari 9.1.3 (11601.7.8)
Actions #6

Updated by Go MAEDA over 7 years ago

  • Target version set to 3.4.0
Actions #7

Updated by Jean-Philippe Lang over 7 years ago

  • Status changed from New to Needs feedback

Could we fix the style without having to change the DOM element from input to button?

Actions #8

Updated by Marius BĂLTEANU over 7 years ago

Jean-Philippe Lang wrote:

Could we fix the style without having to change the DOM element from input to button?

Attached an alternative patch that fixes this issue only from CSS. I chose to target all input elements with type submit to prevent further issues like this.

Actions #9

Updated by Jean-Philippe Lang over 7 years ago

  • Assignee set to Go MAEDA

Thanks for the patch. Go, can you confirm the fix works for you?

Actions #10

Updated by Go MAEDA over 7 years ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Go MAEDA to Jean-Philippe Lang

Jean-Philippe Lang wrote:

Thanks for the patch. Go, can you confirm the fix works for you?

Thanks to Marius BALTEANU, the fix works completely for me.

Actions #11

Updated by Jean-Philippe Lang over 7 years ago

  • Status changed from Resolved to Closed
  • Target version deleted (3.4.0)
  • Resolution set to Fixed

Thanks for your feedback, the patch is committed.

Actions

Also available in: Atom PDF