Skip to content

Feature/table popover dist #1

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.

Sign up for GitHub

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

Jump to bottom
Open
reinaldocoelho wants to merge 161 commits into
base: feature/table-popover-dist
Choose a base branch
from
Open

Feature/table popover dist #1

reinaldocoelho wants to merge 161 commits into from

Conversation

@reinaldocoelho
Copy link

@reinaldocoelho reinaldocoelho commented Oct 13, 2016 •
edited
Loading

What does this PR do?

  • Changed icons of actions on table: Add row, Add column and etc.
  • Included language support for table actions.
  • Included translation to pt_BR.
  • Inproved line functions to copy style from original element.
  • Corrected bug when action is applied on table with other table inner.

Where should the reviewer start?

  • start on the src/js/bs3/settings.js
  • start on the src/js/bs3/module/Buttons.js
  • start on the src/js/base/summernote-en-US.js
  • start on the lang/summernote-pt-BR.js
  • start on the src/js/base/editing/Table.js

How should this be manually tested?

  • Create any table on summernote, then add, remove and change columns and rows.

Screenshots (if for frontend)

image

Checklist

  • verify icons to actions
  • verify the actions with formatted tables
  • verify the actions in table with other table inner

mstred and others added 30 commits May 16, 2016 16:27
@mstred
Add support to custom styletag styles and formatting
ae2bac3
@mstred
Add note-styletag validation in paragraph insertion
64cc4c6
@sionide21
Remove invalid pointer-event css property
f95a6cf
@fgallinari
Update summernote-pt-BR.js
1565cd2 
Help strings added, plus other minor changes
Add isSupportTouch on agent.js
ec8f386
Add tooltip in settings and make it automatically applied by agent.is…
2d6d1a6 
…SupportTouch
@lqez
Merge pull request summernote#1914 from fgallinari/patch-1
5481f52 
Update summernote-pt-BR.js
@lqez
Revert "Update summernote-pt-BR.js"
60bbc8d
@lqez
Merge pull request summernote#1918 from summernote/revert-1914-patch-1
d2956d7 
Revert "Update summernote-pt-BR.js"
Refactoring for options
ab1de58
@dany28
Adding paste event to LinkDialog
50a4e6b 
When user paste url, link text is changing but button is still disabled.
Adding paste event fixing it
fix drop zone memory leak bug
0b764b5
add semicolon
c091159
@dodo360
Update README.md
8624d23 
Added link to adaptation for react
@hackerwins
Merge pull request summernote#1968 from Vnkitaev/patch-1
de9d422 
Update README.md
@hackerwins
move location for react-summernote
f477432
@hackerwins
Merge pull request summernote#1951 from kbs0327/fix_memory_leak
54ef94d 
fix memory leak bug in drop zone
@hackerwins
fixed typo.
7e9b4ad
@hackerwins
Merge pull request summernote#1896 from SalesLoft/css-pointer-events
4c3f5a0 
Remove invalid `pointer-event` css property
@hackerwins
Merge pull request summernote#1920 from dany28/develop
18bbe07 
Adding paste event to LinkDialog
@hackerwins
release 'paste' event when dialog is closed. related with $1920.
d0894e8
@avif
Fixed summernote#1977.
669f35b
add license
c0c2e50
@justinyoo
Add .nuspec for NuGet packaging
997db57
@justinyoo
Update .nuspec to fall back locating content files
b0d5a5a
@hackerwins
Merge pull request summernote#1981 from graingert/add-license
a039c1c 
add license
@hackerwins
Comment out IE9 test
1089880
@hackerwins
Merge branch 'develop' of https://github.com/summernote/summernote in…
f6c68e1 
…to develop
@hackerwins
update dist files
414718c
@hackerwins
Comment out Edge
cc6fb1b
hackerwins and others added 30 commits May 13, 2017 20:28
@hackerwins
Merge pull request summernote#2320 from IonutVornicescu/patch-1
25c6b0b 
Correct exit from fullscreen
@hackerwins
Extract resizeTo method from toggle method. related with summernote#2320
6894e17 
.
@hackerwins
Prevent removing another event callback on window/document. related w…
efe8ea3 
…ith summernote#2320.
@hackerwins
Fixed wrong position of checkbox after merging summernote#2323.
91986f6
@hackerwins
Merge pull request summernote#2195 from dmitriy-strukov/add-target-blank
444e989 
Feature | Added target blank option
@hackerwins
Prevent overwriting default link target option with editing state. re…
4837727 
…lated with summernote#2195.
@hackerwins
Merge pull request summernote#2160 from czimnick/patch-1
b43f52b 
fix for outsideris/angular-summernote#162
@hackerwins
added comments.
7ffcdc8
@hackerwins
Merge pull request summernote#1917 from summernote/minor/disable-tool…
8d0cf29 
…tip-by-option

Tooltip can be shown/hidden by settings
@hackerwins
Please don't include dist/* files on your commits.
b757a66
@hackerwins
Merge pull request summernote#2086 from latifz1/develop
2ebc56c 
508 compliance
@hackerwins
Merge branch 'develop' into bugfix/summernote#1674
ec92812
@hackerwins
Merge pull request summernote#1919 from mstred/feature/custom_styletags
e2173ec 
Add support to custom styletags right into the style toolbar menu
@hackerwins
Skip buttons.spec tests on firefox.
3c1350e
@hackerwins
Merge branch 'develop' of github.com:summernote/summernote into develop
1c6011c
@hackerwins
Skip buttons.spec tests on Edge.
08d03d9
@reinaldocoelho
Merge branch 'develop' into feature/table-popover-dist
4415722 
# Conflicts:
#	dist/font/summernote.eot
#	dist/font/summernote.ttf
#	dist/font/summernote.woff
#	dist/lang/summernote-ar-AR.min.js
#	dist/lang/summernote-bg-BG.min.js
#	dist/lang/summernote-ca-ES.min.js
#	dist/lang/summernote-cs-CZ.min.js
#	dist/lang/summernote-da-DK.min.js
#	dist/lang/summernote-de-DE.min.js
#	dist/lang/summernote-es-ES.min.js
#	dist/lang/summernote-es-EU.min.js
#	dist/lang/summernote-fa-IR.min.js
#	dist/lang/summernote-fi-FI.min.js
#	dist/lang/summernote-fr-FR.js
#	dist/lang/summernote-fr-FR.min.js
#	dist/lang/summernote-gl-ES.min.js
#	dist/lang/summernote-he-IL.min.js
#	dist/lang/summernote-hr-HR.min.js
#	dist/lang/summernote-hu-HU.min.js
#	dist/lang/summernote-id-ID.min.js
#	dist/lang/summernote-it-IT.min.js
#	dist/lang/summernote-ja-JP.min.js
#	dist/lang/summernote-ko-KR.min.js
#	dist/lang/summernote-lt-LT.min.js
#	dist/lang/summernote-lt-LV.min.js
#	dist/lang/summernote-nb-NO.min.js
#	dist/lang/summernote-nl-NL.min.js
#	dist/lang/summernote-pl-PL.min.js
#	dist/lang/summernote-pt-BR.min.js
#	dist/lang/summernote-pt-PT.min.js
#	dist/lang/summernote-ro-RO.min.js
#	dist/lang/summernote-ru-RU.min.js
#	dist/lang/summernote-sk-SK.min.js
#	dist/lang/summernote-sl-SI.min.js
#	dist/lang/summernote-sr-RS-Latin.min.js
#	dist/lang/summernote-sr-RS.min.js
#	dist/lang/summernote-sv-SE.min.js
#	dist/lang/summernote-th-TH.min.js
#	dist/lang/summernote-tr-TR.js
#	dist/lang/summernote-tr-TR.min.js
#	dist/lang/summernote-uk-UA.min.js
#	dist/lang/summernote-vi-VN.min.js
#	dist/lang/summernote-zh-CN.min.js
#	dist/lang/summernote-zh-TW.min.js
#	dist/plugin/databasic/summernote-ext-databasic.min.js
#	dist/plugin/hello/summernote-ext-hello.min.js
#	dist/plugin/specialchars/summernote-ext-specialchars.min.js
#	dist/summernote.css
#	dist/summernote.js
#	dist/summernote.min.js
@reinaldocoelho
New dist created after merge and conflict solve with V0.8.3.
c6e7a40
@reinaldocoelho
Removing dist files.
0d8b112
@reinaldocoelho
Adding unit tests to table process
5595cb3
@reinaldocoelho
Adding support to remove table column with colspan attribute
e8a6b6e
@reinaldocoelho
Adding support to remove table row with columns with rowspan attribute
fd92f93
@reinaldocoelho
Change table structure to process Col delete with colspan and rowspan.
96d9e32
@reinaldocoelho
change table work with rowspan and colspan for delete col and row.
9c7d233
@reinaldocoelho
Adding test case for delete row with virtual span cell.
335be6a
@reinaldocoelho
change add test to insert row on top of rowspan
dd12ec4
@reinaldocoelho
Solve bug when add line bellow rowspan
05d8573
@reinaldocoelho
Add suport to add column when have rowspan in table
41ae580
@reinaldocoelho
change to test remove column after colspan
320133f
@reinaldocoelho
change to test add column before colspan column
fa54161
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.