Skip to content

[stable12] Fix edit tag textbox size - #8408

Merged
MorrisJobke merged 1 commit into
stable12from
stable12-8376-devgals
Feb 17, 2018
Merged

[stable12] Fix edit tag textbox size#8408
MorrisJobke merged 1 commit into
stable12from
stable12-8376-devgals

Conversation

@AasthaGupta

Copy link
Copy Markdown
Member

Backport of #8376

Fixes#7586
Signed-off-by: Aastha Gupta <aastha.gupta4104@gmail.com>
@MorrisJobkeMorrisJobke added the 3. to review Waiting for reviews label Feb 17, 2018
@MorrisJobkeMorrisJobke added this to the Nextcloud 12.0.6 milestone Feb 17, 2018
@codecov

codecovBot commented Feb 17, 2018

Copy link
Copy Markdown

Codecov Report

Merging #8408 into stable12 will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## stable12 #8408 +/- ##
===========================================
Coverage 53.84% 53.84% Complexity 22755 22755 ===========================================
Files 1386 1386 Lines 87129 87129 Branches 1331 1331 ===========================================
Hits 46915 46915 Misses 40214 40214
Impacted FilesCoverage ΔComplexity Δ
lib/private/Server.php84.68% <0%> (-0.13%)121% <0%> (ø)
lib/private/Security/CertificateManager.php92.07% <0%> (+0.99%)39% <0%> (ø)⬇️

@danxuliudanxuliu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

However, I have noticed that the final height of the input field is 32px instead of 30px; this is caused by the min-height rule applied to input elements in the server. Anyway, works for me ;-)

@MorrisJobke
MorrisJobke merged commit 3ab9919 into stable12Feb 17, 2018
@MorrisJobke
MorrisJobke deleted the stable12-8376-devgals branch February 17, 2018 14:38
@MorrisJobkeMorrisJobke mentioned this pull request Mar 9, 2018
2 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@AasthaGupta@MorrisJobke@danxuliu