-
Notifications
You must be signed in to change notification settings - Fork 131
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
[1/5] Defining options for the text widget. #32
Conversation
Each set of options applies to a range of characters equal to the length of the text added. E.g. on write calls like: Write("text", opts1) Write("long text", opts2) The two passed options will be stored as: opts1 for byte range low:0, high:4 opts2 for byte range low:4, high:13 The text itself will be stored in a bytes.Buffer.
The line scanner identifies and counts lines in the input text and performs wrapping if required.
And adding a demo.
Pull Request Test Coverage Report for Build 67
💛 - Coveralls |
// See the License for the specific language governing permissions and | ||
// limitations under the License. | ||
|
||
package text |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please provide a documentation for the package. Maybe it makes sense to move the comment related to the file on line 22 here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, this is related to how I split this into multiple pull requests. The documentation is in a later pull request, in the main file for this package.
- renaming it to cvsWidth. - adding a comment explaining that it is the width of the canvas.
[5/5] Implementing the text widget.
[4/5] Adding an object that tracks the scrolling state.
[3/5] Adding line scanner.
[2/5] Defining options for text written to the widget.
Works on #4