Ticket #1514 (closed defect: fixed)
mcedit: optional different background for columns > 80
| Reported by: | sergey-feo | Owned by: | angel_il |
|---|---|---|---|
| Priority: | major | Milestone: | 4.7.1 |
| Component: | mcedit | Version: | master |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Branch state: | Votes for changeset: | committed-master |
Description
In programming good style is do not exceed 80 columns.
It will be good if mcedit will have option for different background color for columns 81 and righter.
You can see examples in attached files.
Attachments
Change History
comment:1 in reply to: ↑ description Changed 15 years ago by angel_il
comment:2 Changed 15 years ago by angel_il
- Status changed from new to accepted
- Owner set to angel_il
comment:4 Changed 15 years ago by sergey-feo
are you really need this?
I can live without this, but with this feature life is better :-)
Please note that old Borland C++ Builder 6 also have feature like this.
comment:6 Changed 15 years ago by NNemec
I agree that this would be a really helpful feature. I am working on a project where the 80char limit is strict policy. I don't know how often a simple visual help like the one suggested would have helped preventing policy violations.
comment:8 Changed 15 years ago by angel_il
branch: 1514_over_wrap_line
changeset: aafd9353ab45a00c32ff90a52c6235a6caec5631
comment:9 Changed 15 years ago by angel_il
- severity changed from no branch to on review
- Milestone changed from 4.7 to 4.7.1
changeset: ef4b02eeda68d94c18d96087e1f23526ca8719c8
please review
comment:11 Changed 15 years ago by slavazanko
- Votes for changeset changed from andrew_b to andrew_b slavazanko
- severity changed from on review to approved
comment:12 Changed 15 years ago by angel_il
- Status changed from accepted to testing
- Votes for changeset changed from andrew_b slavazanko to committed-master
- Resolution set to fixed
- severity changed from approved to merged
comment:14 Changed 15 years ago by angel_il
- Status changed from closed to reopened
- Votes for changeset committed-master deleted
- Resolution fixed deleted
comment:15 Changed 15 years ago by angel_il
reason: http://mail.gnome.org/archives/mc-devel/2010-January/msg00035.html
One with curtsor position at 71'th character, another at 72'th position.
The background suddenly becomes black.
--
Janek Kozicki
screenshot: http://mail.gnome.org/archives/mc-devel/2010-January/pngytUgAUOM8q.png
comment:16 Changed 15 years ago by angel_il
- severity changed from merged to on review
branch: 1514_right_margin_fix
changeset: b8ee28c2fd806c2ae6a9d1e813b29bb74f285238
comment:17 Changed 15 years ago by andrew_b
- Votes for changeset set to andrew_b
- Version changed from 4.7.0-pre1 to master
- Type changed from enhancement to defect
comment:18 Changed 15 years ago by metux
rebased and running through testfarm.
i'll vote if it passes all build-tests.
comment:19 Changed 15 years ago by slavazanko
- Status changed from reopened to assigned
- Votes for changeset changed from andrew_b to andrew_b slavazanko
- severity changed from on review to approved
comment:20 Changed 15 years ago by metux
- Votes for changeset changed from andrew_b slavazanko to committed-master
- severity changed from approved to merged
comment:21 Changed 15 years ago by metux
- Status changed from assigned to testing
- Resolution set to fixed
qt designer with 80 columns delimiter