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

fix: reshuffle __newname field to accommodate under 1st Tab Break #18406

Merged
merged 2 commits into from Oct 17, 2022

Conversation

saurabh6790
Copy link
Member

@saurabh6790 saurabh6790 commented Oct 13, 2022

If doctype contains Tab Breaks and the auto naming field is set to Prompt, then the __newname field gets rendered in a different section outside the Tab Structure.

Before Fix

Screenshot 2022-10-13 at 11 42 30 AM

New section for __newname field
Screenshot 2022-10-13 at 11 42 40 AM

After Fix

__newfield get rendered under 1st Tab Break

Screenshot 2022-10-13 at 11 39 26 AM

@saurabh6790 saurabh6790 requested a review from a team as a code owner October 13, 2022 06:44
@saurabh6790 saurabh6790 requested review from shariquerik and removed request for a team October 13, 2022 06:44
@saurabh6790 saurabh6790 changed the title ['fix: reshuffle __newname field to accomodate under 1st Tab Break ['fix: reshuffle __newname field to accommodate under 1st Tab Break Oct 13, 2022
@codecov
Copy link

codecov bot commented Oct 13, 2022

Codecov Report

Merging #18406 (73cbb46) into develop (241f270) will increase coverage by 0.14%.
The diff coverage is 60.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #18406      +/-   ##
===========================================
+ Coverage    62.48%   62.62%   +0.14%     
===========================================
  Files          743      743              
  Lines        67016    67020       +4     
  Branches      5960     5961       +1     
===========================================
+ Hits         41873    41972      +99     
+ Misses       21603    21548      -55     
+ Partials      3540     3500      -40     
Flag Coverage Δ
server-ui 29.86% <ø> (+<0.01%) ⬆️
ui-tests 49.05% <60.00%> (+0.38%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@sagarvora sagarvora changed the title ['fix: reshuffle __newname field to accommodate under 1st Tab Break fix: reshuffle __newname field to accommodate under 1st Tab Break Oct 13, 2022
@ankush ankush added the backport version-14-hotfix backport to version 14 label Oct 17, 2022
@ankush ankush merged commit 35995f0 into frappe:develop Oct 17, 2022
mergify bot pushed a commit that referenced this pull request Oct 17, 2022
…8406)

* fix: reshuffle __newname field to accommodate under 1st Tab Break

* fix: handle empty condition

(cherry picked from commit 35995f0)
ankush pushed a commit that referenced this pull request Oct 17, 2022
…8406) (#18427)

* fix: reshuffle __newname field to accommodate under 1st Tab Break

* fix: handle empty condition

(cherry picked from commit 35995f0)

Co-authored-by: Saurabh <saurabh6790@gmail.com>
frappe-pr-bot pushed a commit that referenced this pull request Oct 18, 2022
# [14.12.0](v14.11.1...v14.12.0) (2022-10-18)

### Bug Fixes

* add flags to set_permission for docshares ([#18416](#18416)) ([#18429](#18429)) ([a2ba0aa](a2ba0aa))
* correct import for markupsafe.escape ([c2f1c57](c2f1c57))
* **dx:** resolve_class ([#18417](#18417)) ([#18421](#18421)) ([c47de0d](c47de0d))
* explicitly set doctype in queries ([#18403](#18403)) ([#18405](#18405)) ([6848ee9](6848ee9))
* format currency value as float while grid upload ([5eb803c](5eb803c))
* google calendar sync times ([#18384](#18384)) ([#18394](#18394)) ([7c9ee6d](7c9ee6d))
* Notifications. Get Alerts for Today button [#18423](#18423) ([#18428](#18428)) ([63e3c61](63e3c61))
* progress bar not disappearing ([#18375](#18375)) ([#18392](#18392)) ([e0d6d0b](e0d6d0b))
* reshuffle __newname field to accommodate under 1st Tab Break ([#18406](#18406)) ([#18427](#18427)) ([f833e5d](f833e5d))
* Show error page even when routing fails ([#18437](#18437)) ([#18438](#18438)) ([5d3e97a](5d3e97a))
* **UI:** child table custom horizontal scroll ([ec09123](ec09123))
* **UI:** Child table responsive ([cb866f9](cb866f9))
* **UI:** consider 2px border at grid end ([c85fb2d](c85fb2d))
* **UI:** lesser margin if cell is focused ([f3ad63a](f3ad63a))

### Features

* allow syncing new fields in Doctype Layout ([#18408](#18408)) ([c73e2e2](c73e2e2))
* search in translated title, if we show title ([#17828](#17828)) ([#18395](#18395)) ([73758d7](73758d7))

### Performance Improvements

* Perform `db.set_value` with single query only (backport [#18305](#18305)) ([#18349](#18349)) ([23caa1f](23caa1f))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport version-14-hotfix backport to version 14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants