Skip to content

Replace Simulate Query dialog with a modal#17065

Merged
MauricioFauth merged 1 commit intophpmyadmin:masterfrom
shucontech:simulate_query_modal
Sep 16, 2021
Merged

Replace Simulate Query dialog with a modal#17065
MauricioFauth merged 1 commit intophpmyadmin:masterfrom
shucontech:simulate_query_modal

Conversation

@shucontech
Copy link
Copy Markdown
Contributor

Signed-off-by: Saksham Gupta shucontech@gmail.com

Description

Replace Simulate Query dialog with a modal

Before submitting pull request, please review the following checklist:

  • Make sure you have read our CONTRIBUTING.md document.
  • Make sure you are making a pull request against the correct branch. For example, for bug fixes in a released version use the corresponding QA branch and for new features use the master branch. If you have a doubt, you can ask as a comment in the bug report or on the mailing list.
  • Every commit has proper Signed-off-by line as described in our DCO. This ensures that the work you're submitting is your own creation.
  • Every commit has a descriptive commit message.
  • Every commit is needed on its own, if you have just minor fixes to previous commits, you can squash them.
  • Any new functionality is covered by tests.

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 9, 2021

Codecov Report

Merging #17065 (b58516e) into master (c90affe) will decrease coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #17065      +/-   ##
============================================
- Coverage     47.76%   47.63%   -0.13%     
- Complexity    16204    16544     +340     
============================================
  Files           490      571      +81     
  Lines         61956    60233    -1723     
============================================
- Hits          29592    28694     -898     
+ Misses        32364    31539     -825     
Flag Coverage Δ
arch-7-amd64 ?
arch-7-arm32v6 ?
arch-7-i386 ?
arch-7-ppc64le ?
arch-7-s390x ?
dbase-extension 47.38% <ø> (-0.05%) ⬇️
recode-extension 47.31% <ø> (+0.02%) ⬆️
unit-7.2-ubuntu-latest 47.40% <ø> (+0.07%) ⬆️
unit-7.3-ubuntu-latest 50.38% <ø> (+0.35%) ⬆️
unit-7.4-ubuntu-latest 50.32% <ø> (+0.39%) ⬆️
unit-8.0-ubuntu-latest 50.40% <ø> (+0.38%) ⬆️

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

Impacted Files Coverage Δ
...Controllers/Database/MultiTableQueryController.php 0.00% <0.00%> (-62.50%) ⬇️
...sses/Properties/Plugins/ImportPluginProperties.php 50.00% <0.00%> (-50.00%) ⬇️
libraries/classes/Charsets.php 50.90% <0.00%> (-32.73%) ⬇️
libraries/classes/IndexColumn.php 88.63% <0.00%> (-11.37%) ⬇️
libraries/classes/Pdf.php 68.88% <0.00%> (-7.78%) ⬇️
libraries/classes/Plugins/Export/ExportOdt.php 89.20% <0.00%> (-7.30%) ⬇️
libraries/classes/Plugins/Export/ExportOds.php 90.74% <0.00%> (-6.88%) ⬇️
libraries/classes/Plugins/Export/ExportExcel.php 93.93% <0.00%> (-6.07%) ⬇️
.../classes/Controllers/Table/StructureController.php 0.00% <0.00%> (-5.61%) ⬇️
libraries/classes/Core.php 63.06% <0.00%> (-4.08%) ⬇️
... and 439 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c90affe...b58516e. Read the comment docs.

Signed-off-by: Saksham Gupta <shucontech@gmail.com>

Set Aria hidden to true

Signed-off-by: Saksham Gupta <shucontech@gmail.com>
@MauricioFauth MauricioFauth merged commit 87b9194 into phpmyadmin:master Sep 16, 2021
@MauricioFauth MauricioFauth self-assigned this Sep 16, 2021
@MauricioFauth MauricioFauth added this to the 5.2.0 milestone Sep 16, 2021
@MauricioFauth MauricioFauth added the UI Issues relating to the user interface label Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI Issues relating to the user interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants